STENCIL-2493

Correctly hiding the “none” option for required pick lists (without photo)
bigcommerce · Feb 21, 2017 · 09a0573 · 09a0573
1 parent c4f8c14
commit 09a0573
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -13,6 +13,7 @@
 - Fix an issue with special characters in search results for content pages [#933](https://github.com/bigcommerce/stencil/pull/933)
 - Fix an issue with special characters in carousel text [#932](https://github.com/bigcommerce/stencil/pull/932)
 - Remove an unnecessary and confusing option in theme editor [#927](https://github.com/bigcommerce/stencil/pull/927)
+- Fix an issue where required product list options would display an invalid "none" choice [#929](https://github.com/bigcommerce/stencil/pull/929)
 
 
 ## 1.5.2 (2017-02-14)

diff --git a/templates/components/products/options/product-list.html b/templates/components/products/options/product-list.html
@@ -43,14 +43,15 @@
             {{/each}}
         </ul>
     {{else}}
+        {{#unless required}}
         <input class="form-radio"
                type="radio"
                name="attribute[{{id}}]"
                value="0"
                id="attribute_0_{{id}}"
                {{#if defaultValue '==' 0}}checked{{/if}}
-               {{#if required}}required{{/if}}>
         <label class="form-label" for="attribute_0_{{id}}">{{lang 'products.none'}}</label>
+        {{/unless}}
         {{#each values}}
             <input
                 class="form-radio"