Merge pull request #85 from ljmotta/quick-fix-select

Fix SelectField

Author: ljmotta

examples/package.json

@@ -19,7 +19,7 @@
     "uniforms-bridge-json-schema": "3.5.1",
     "uniforms-bridge-simple-schema": "3.5.1",
     "uniforms-bridge-simple-schema-2": "3.5.1",
-    "uniforms-patternfly": "4.6.0"
+    "uniforms-patternfly": "4.6.1"
   },
   "devDependencies": {
     "parcel-bundler": "^1.12.5",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "uniforms-patternfly",
-  "version": "4.6.0",
+  "version": "4.6.1",
   "description": "Patternfly forms for uniforms",
   "repository": "[email protected]:aerogear/uniforms-patternfly.git",
   "author": "Gianluca <[email protected]>",

src/SelectField.tsx

@@ -171,14 +171,16 @@ function SelectField(props: SelectFieldProps) {
     [props]
   );
 
-  if (props.placeholder)
-    selectedOptions.unshift(
-      <SelectOption
-        key={props.allowedValues!.length}
-        isPlaceholder
-        value={props.placeholder}
-      />
-    );
+  useEffect(() => {
+    if (props.placeholder)
+      selectedOptions.unshift(
+        <SelectOption
+          key={props.allowedValues!.length}
+          isPlaceholder
+          value={props.placeholder}
+        />
+      );
+  }, [props.placeholder, selectedOptions])
 
   return wrapField(
     props,