ZeroGachis · Mibou · Oct 26, 2023 · Oct 26, 2023
diff --git a/.prettierrc b/.prettierrc
@@ -0,0 +1,6 @@
+{
+    "tabWidth": 4,
+    "useTabs": false,
+    "singleQuote": true,
+    "jsxSingleQuote": true
+}
diff --git a/.prettierrc.json b/.prettierrc.json
@@ -1,16 +1,17 @@
 {
-  "singleQuote": true,
-  "semi": true,
-  "printWidth": 100,
-  "tabWidth": 4,
-  "trailingComma": "all",
-  "endOfLine": "lf",
-  "overrides": [
-    {
-      "files": "*.json",
-      "options": {
-        "tabWidth": 2
-      }
-    }
-  ]
+    "singleQuote": true,
+    "semi": true,
+    "printWidth": 100,
+    "tabWidth": 4,
+    "trailingComma": "all",
+    "endOfLine": "lf",
+    "jsxSingleQuote": true,
+    "overrides": [
+        {
+            "files": "*.json",
+            "options": {
+                "tabWidth": 2
+            }
+        }
+    ]
 }
diff --git a/Storybook/components/NumberField/NumberField.stories.tsx b/Storybook/components/NumberField/NumberField.stories.tsx
@@ -1,5 +1,5 @@
 import type { Meta, StoryObj } from '@storybook/react-native';
-import React from 'react';
+import React, { useState } from 'react';
 import { StyleSheet, View } from 'react-native';
 import { NumberField } from 'smartway-react-native-ui';
 
@@ -9,9 +9,9 @@ export default {
     title: 'components/NumberField',
     component: NumberField,
     args: {
-        value: '0',
-        minValue: 0,
-        maxValue: 10,
+        value: '-999.9',
+        minValue: -999.9,
+        maxValue: 999.9,
     },
     argTypes: {
         state: {
@@ -26,6 +26,7 @@ export default {
             ],
         },
         size: { control: { type: 'radio' }, options: ['m', 's'] },
+        decimal: { control: { type: 'radio' }, options: [true, false] },
     },
 
     decorators: [
@@ -44,7 +45,25 @@ export default {
 
 type Story = StoryObj<ComponentProps>;
 
+const NumberFieldTester = (args) => {
+    const [quantity, setQuantity] = useState<number>(args.value);
+    const onValueChange = (newQuantity: number) => {
+        setQuantity(newQuantity);
+    };
+    return (
+        <NumberField
+            minValue={args.minValue}
+            maxValue={args.maxValue}
+            size={args.size}
+            decimal={args.decimal}
+            value={quantity}
+            onValueChange={onValueChange}
+        />
+    );
+};
 export const Default: Story = {
-    args: {},
+    render: (args) => {
+        return NumberFieldTester(args);
+    },
 };
 Default.parameters = { noSafeArea: false };
diff --git a/Storybook/components/NumberSelector/NumberSelector.stories.tsx b/Storybook/components/NumberSelector/NumberSelector.stories.tsx
@@ -1,5 +1,5 @@
 import type { Meta, StoryObj } from '@storybook/react-native';
-import React from 'react';
+import React, { useState } from 'react';
 import { StyleSheet, View } from 'react-native';
 import { NumberSelector } from 'smartway-react-native-ui';
 
@@ -10,9 +10,12 @@ export default {
     component: NumberSelector,
     args: {
         value: 0,
+        minValue: -999,
+        maxValue: 999,
     },
     argTypes: {
         onValueChange: { action: 'onValueChange' },
+        decimal: { control: { type: 'radio' }, options: [true, false] },
     },
 
     decorators: [
@@ -31,13 +34,28 @@ export default {
 
 type Story = StoryObj<ComponentProps>;
 
+const NumberSelectorTester = (args) => {
+    const [quantity, setQuantity] = useState<number>(args.value);
+    const onValueChange = (newQuantity: number) => {
+        setQuantity(newQuantity);
+    };
+    return (
+        <NumberSelector
+            showSoftInputOnFocus={args.showSoftInputOnFocus}
+            decimal={args.decimal}
+            minValue={args.minValue}
+            maxValue={args.maxValue}
+            minusIcon={args.minusIcon}
+            plusIcon={args.plusIcon}
+            value={quantity}
+            onValueChange={onValueChange}
+        />
+    );
+};
+
 export const Default: Story = {
-    args: {
-        showSoftInputOnFocus: true,
-        minValue: 0,
-        maxValue: 999,
-        minusIcon: 'arrow-back',
-        plusIcon: 'arrow-forward',
+    render: (args) => {
+        return NumberSelectorTester(args);
     },
 };
 Default.parameters = { noSafeArea: false };
diff --git a/src/components/numberField/NumberField.tsx b/src/components/numberField/NumberField.tsx
@@ -1,13 +1,21 @@
+/* eslint-disable no-useless-escape */
 import React, { useEffect, useState } from 'react';
 import { StyleSheet, TextInput, TextInputBase } from 'react-native';
 import { useTheme } from '../../styles/themes';
 
 type FieldBaseProps = React.ComponentProps<typeof TextInputBase>;
 export interface NumberFieldProps extends FieldBaseProps {
-    state?: 'readonly' | 'filled' | 'prefilled' | 'filled-focused' | 'prefilled-focused' | 'error';
+    state?:
+        | 'readonly'
+        | 'filled'
+        | 'prefilled'
+        | 'filled-focused'
+        | 'prefilled-focused'
+        | 'error';
     size?: 'm' | 's';
     minValue?: number;
     maxValue?: number;
+    decimal?: boolean;
 }
 
 export const NumberField = React.forwardRef<TextInput, NumberFieldProps>(
@@ -17,21 +25,30 @@
             size = 'm',
             minValue = 0,
             maxValue = 999,
+            decimal = false,
             ...props
         }: NumberFieldProps,
-        ref,
+        ref
     ) => {
         const theme = useTheme();
-
+        const decimalRegex =
+            minValue !== undefined && minValue < 0
+                ? /^-?\d+[\.]?\d?$/
+                : /^\d+[\.]?\d?$/;
+        const integerRegex =
+            minValue !== undefined && minValue < 0 ? /^-?\d+$/ : /^\d+$/;
+        const numberRegex = decimal ? decimalRegex : integerRegex;
         const [currentState, setCurrentState] = useState<string>(state);
         const [filled, setFilled] = useState<boolean>(false);
         const [error, setError] = useState<boolean>(false);
         const [focused, setFocused] = useState<boolean>(false);
         const [forcedState, setForcedState] = useState<boolean>(true);
-        const [firstContentChange, setFirstContentChange] = useState<boolean>(true);
+        const [firstContentChange, setFirstContentChange] =
+            useState<boolean>(true);
         const [firstValue, setFirstValue] = useState<string>();
         const [value, setValue] = useState<string>(props.value ?? '');
         const [lastValue, setLastValue] = useState<string>();
+        const parser = decimal ? parseFloat : parseInt;
 
         useEffect(() => {
             if (forcedState) {
@@ -39,7 +56,7 @@
                return;
            }
            refreshCurrentState();
        }, [error, filled, focused]);

        useEffect(() => {
            if (firstContentChange) {
@@ -48,9 +65,9 @@
                 return;
             }
             setFilled(props.value !== firstValue);
-            if (cleanContent(props.value) !== '') setLastValue(value);
+            if (numberRegex.test(props.value ?? '')) setLastValue(value);
             checkContent(props.value);
         }, [props.value]);
 
        useEffect(() => {
            if (firstContentChange) {
@@ -59,7 +76,7 @@
                 return;
             }
             setFilled(value !== firstValue);
-            if (cleanContent(value) !== '') {
+            if (numberRegex.test(props.value ?? '')) {
                 setLastValue(value);
             }
             checkContent(value);
@@ -73,7 +90,8 @@
                 case 'prefilled':
                     textColor = theme.sw.colors.neutral[500];
                     borderColor = undefined;
-                    backgroundColor = theme.sw.colors.neutral[500] + theme.sw.transparency[8];
+                    backgroundColor =
+                        theme.sw.colors.neutral[500] + theme.sw.transparency[8];
                     break;
                 case 'filled-focused':
                     textColor = theme.sw.colors.neutral[800];
@@ -83,17 +101,21 @@
                 case 'prefilled-focused':
                     textColor = theme.sw.colors.primary.main;
                     borderColor = theme.sw.colors.primary.main;
-                    backgroundColor = theme.sw.colors.primary.main + theme.sw.transparency[16];
+                    backgroundColor =
+                        theme.sw.colors.primary.main +
+                        theme.sw.transparency[16];
                     break;
                 case 'filled':
                     textColor = theme.sw.colors.neutral[800];
                     borderColor = undefined;
-                    backgroundColor = theme.sw.colors.neutral[500] + theme.sw.transparency[8];
+                    backgroundColor =
+                        theme.sw.colors.neutral[500] + theme.sw.transparency[8];
                     break;
                 case 'error':
                     textColor = theme.sw.colors.error.main;
                     borderColor = undefined;
-                    backgroundColor = theme.sw.colors.error.main + theme.sw.transparency[8];
+                    backgroundColor =
+                        theme.sw.colors.error.main + theme.sw.transparency[8];
                     break;
                 case undefined:
                     break;
@@ -106,7 +128,8 @@
                     borderWidth: borderColor !== undefined ? 1 : 0,
                     borderColor: borderColor,
 
-                    width: size === 's' ? 43 : 72,
+                    width:
+                        size === 's' ? (decimal ? 63 : 43) : decimal ? 110 : 72,
 
                     color: textColor,
                     fontStyle: 'normal',
@@ -132,45 +155,50 @@
                 else setCurrentState('prefilled');
             }
         };
+
         const checkContent = (text: string | undefined) => {
-            if (text !== undefined && text !== '') {
-                const cleanNumber = text.replace(/[^0-9]/g, '');
-                const parsedValue = parseInt(cleanNumber);
-                if (parsedValue !== undefined) {
+            if (
+                text !== undefined &&
+                text !== '' &&
+                ((minValue !== undefined && minValue < 0 && text !== '-') ||
+                    minValue === undefined ||
+                    (minValue !== undefined && minValue >= 0))
+            ) {
+                const parsedValue = parser(text);
+                if (!Number.isNaN(parsedValue)) {
                     setError(
                         (minValue !== undefined && parsedValue < minValue) ||
-                            (maxValue !== undefined && parsedValue >= maxValue),
+                            (maxValue !== undefined && parsedValue >= maxValue)
                     );
                 }
             }
         };
-        const cleanContent = (text: string | undefined) => {
-            if (text !== undefined && text !== '') {
-                const cleanNumber = text.replace(/[^-0-9]/g, '');
-                const parsedValue = parseInt(cleanNumber);
-                return parsedValue.toString();
-            }
-            return '';
-        };
         const onChangeText = (e: any) => {
             if (props?.onChangeText !== undefined) {
                 props.onChangeText(e);
                 checkContent(props.value);
             } else {
-                if (e == '') setValue('');
-                else if (cleanContent(e) != '') setValue(cleanContent(e));
+                if (e == '' || (allowedMinus() && e == '-')) setValue(e);
+                else if (numberRegex.test(e)) setValue(e);
                 checkContent(value);
             }
         };
         const onFocus = (e: any) => {
             setFocused(true);
             if (props?.onFocus !== undefined) props.onFocus(e);
         };
+        const allowedMinus = (): boolean => {
+            return minValue !== undefined && minValue < 0;
+        };
         const onBlur = (e: any) => {
             setFocused(false);
-            if ((value === '' || props.value === '') && firstValue !== '') onChangeText(firstValue);
-            else if ((value === '' || props.value === '') && lastValue !== '') {
-                onChangeText(lastValue);
+            if (
+                value === '' ||
+                props.value === '' ||
+                (allowedMinus() && (value === '-' || props.value === '-'))
+            ) {
+                if (firstValue !== '') onChangeText(firstValue);
+                else if (lastValue !== '') onChangeText(lastValue);
             }
             if (props?.onBlur !== undefined) props.onBlur(e);
         };
@@ -185,14 +213,16 @@
                 onChangeText={(e) => onChangeText(e)}
                 onBlur={(e) => onBlur(e)}
                 onFocus={(e) => onFocus(e)}
-                selectionColor={theme.sw.colors.primary.main + theme.sw.transparency[16]}
+                selectionColor={
+                    theme.sw.colors.primary.main + theme.sw.transparency[16]
+                }
                 cursorColor={theme.sw.colors.primary.main}
-                keyboardType="number-pad"
+                keyboardType='number-pad'
                 editable={state !== 'readonly'}
                 textAlign={'center'}
             />
         );
-    },
+    }
 );
 
 NumberField.displayName = 'NumberField';