fix: mark input fields in options as required

foray1010 · Mar 3, 2023 · 21664c5 · 21664c5
1 parent 025e8cc
commit 21664c5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.size-limit.json b/.size-limit.json
@@ -13,7 +13,7 @@
   },
   {
     "path": "build/production/js/options.js",
-    "limit": "5.34 kB"
+    "limit": "5.37 kB"
   },
   {
     "path": "build/production/js/popup.js",

diff --git a/src/options/components/OptionForm/OptionItem/index.tsx b/src/options/components/OptionForm/OptionItem/index.tsx
@@ -78,6 +78,7 @@ export default function OptionItem(props: Props) {
           maximum={props.maximum}
           minimum={props.minimum}
           name={props.name}
+          required
           value={props.value}
           onBlur={props.onBlur}
           onChange={(evt) => {
@@ -93,6 +94,7 @@ export default function OptionItem(props: Props) {
         <SelectString
           choices={props.choices}
           name={props.name}
+          required
           value={props.value}
           onBlur={props.onBlur}
           onChange={(evt) => {
@@ -117,6 +119,7 @@ export default function OptionItem(props: Props) {
         <InputSelect
           choices={props.choices}
           name={props.name}
+          required
           value={props.value}
           onBlur={(evt) => {
             onChange(evt)