Merge pull request #9 from mprzodala/selectFieldFix

Select field fix
rstgroup · Apr 10, 2017 · 6183005 · 6183005
2 parents da6028b + 713a1c9
commit 6183005
Show file tree

Hide file tree

Showing 6 changed files with 33 additions and 9 deletions.
diff --git a/Bootstrap.js b/Bootstrap.js
diff --git a/Separate.js b/Separate.js
diff --git a/main.js b/main.js
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "react-components-form",
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "React form components",
   "main": "main.js",
   "jsnext:main": "src/index.js",

diff --git a/src/components/SelectField.jsx b/src/components/SelectField.jsx
@@ -28,8 +28,8 @@ export const SelectField = ({
         >
             {options.map(option => (
                 <option
-                    value={option.value || option}
-                    key={option.value || option}
+                    value={option.label ? option.value : option}
+                    key={option.label || option}
                 >
                     {option.label || option}
                 </option>

diff --git a/src/demo/book/index.jsx b/src/demo/book/index.jsx
@@ -34,6 +34,27 @@ const personSchema = new Schema({
     },
     surname: {
         type: String
+    },
+    age: {
+        type: String,
+        options: [
+            {
+                label: 'Select age range',
+                value: ''
+            },
+            {
+                label: '18-21',
+                value: '18-21'
+            },
+            {
+                label: '22-28',
+                value: '22-28'
+            },
+            {
+                label: '29-40',
+                value: '29-40'
+            }
+        ]
     }
 });
 
@@ -84,9 +105,12 @@ const BookForm = () => (
                 <div className={objectFormField}>
                     <TextField name="name" placeholder="name"/>
                 </div>
-                <div>
+                <div className={objectFormField}>
                     <TextField name="surname" placeholder="surname"/>
                 </div>
+                <div>
+                    <SelectField name="age" />
+                </div>
             </ObjectField>
         </ListField>
         <ListField name="languages" label="Languages" className={listWrapper}>