Merge pull request #698 from strapi/fix/numberinput-parser

NumberInput: Fix value assignment during focus
strapi · Sep 20, 2022 · 852cdd8 · 852cdd8
2 parents 084ba39 + 966436e
commit 852cdd8
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/packages/strapi-design-system/src/NumberInput/NumberInput.js b/packages/strapi-design-system/src/NumberInput/NumberInput.js
@@ -50,7 +50,7 @@ export const NumberInput = React.forwardRef(
     const [inputValue, setInputValue] = useState(value === undefined || value === null ? INITIAL_VALUE : String(value));
     const generatedId = useId('numberinput', id);
     const numberParserRef = useRef(new NumberParser(getDefaultLocale()));
-    const numberFormaterRef = useRef(new NumberFormatter(getDefaultLocale(), { maximumSignificantDigits: 21 }));
+    const numberFormaterRef = useRef(new NumberFormatter(getDefaultLocale(), { maximumFractionDigits: 20 }));
 
     const handleChange = (e) => {
       const nextValue = e.target.value;
@@ -143,9 +143,7 @@ export const NumberInput = React.forwardRef(
     };
 
     const handleFocus = () => {
-      if (value !== undefined) {
-        setInputValue(String(numberParserRef.current.format(inputValue) ?? INITIAL_VALUE));
-      }
+      setInputValue(inputValue ?? INITIAL_VALUE);
     };
 
     const handleBlur = () => {