Merge pull request #49 from swalkinshaw/fix-selected-attribute-on-opt…

…ions

Options: only set selected attribute when its true

README.md

@@ -87,9 +87,9 @@ You can also render individual fields using `formative.core/render-field`. Unlik
                 "Vanilla")
 ;; Returns:
 [:select {:name "flavor"}
- ([:option {:value "Chocolate", :selected false} "Chocolate"]
+ ([:option {:value "Chocolate"} "Chocolate"]
   [:option {:value "Vanilla", :selected true} "Vanilla"]
-  [:option {:value "Strawberry", :selected false} "Strawberry"])]
+  [:option {:value "Strawberry"} "Strawberry"])]
 ```
 
 Notice that the "Vanilla" option is selected in our generated element.

src/formative/render.cljc

@@ -137,8 +137,12 @@
     [:textarea attrs (fu/escape-html (render-input-val field))]))
 
 (defn ^:private build-opt-tag [v text val]
-  (let [v (str v)]
-    [:option {:value v :selected (= val v)} text]))
+  (let [v (str v)
+        attrs {:value v}
+        attrs (if (= val v)
+                (assoc attrs :selected "true")
+                attrs)]
+    [:option attrs text]))
 
 (defmethod render-field :select [field]
   (let [attrs (get-input-attrs field [:name :id :class :autofocus