fix: make sure auto identificator gets generated

Co-Authored-By: Andre Luiz Rabello <andre.luiz_r@hotmail.com>
onfido · Feb 9, 2021 · 6795a72 · 6795a72
1 parent 9eeafef
commit 6795a72
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 8 deletions.
diff --git a/packages/react/src/components/input/input.tsx b/packages/react/src/components/input/input.tsx
@@ -18,10 +18,8 @@ export const Input = withRef(
     }: InputProps,
     ref: InputProps['ref']
   ): JSX.Element => {
-    const [autoId] = useState<string | undefined>(() =>
-      children ? `${idPrefix}_${++idCount}` : undefined
-    );
-    const id = externalId || autoId;
+    const [autoId] = useState(() => `${idPrefix}_${++idCount}`);
+    const id = externalId || (children ? autoId : undefined);
 
     return (
       <FieldLabelWrapper id={id}>

diff --git a/packages/react/src/components/textarea/textarea.tsx b/packages/react/src/components/textarea/textarea.tsx
@@ -20,10 +20,8 @@ export const Textarea = withRef(
     }: TextareaProps,
     ref: TextareaProps['ref']
   ): JSX.Element => {
-    const [autoId] = useState<string | undefined>(() =>
-      children ? `${idPrefix}_${++idCount}` : undefined
-    );
-    const id = externalId || autoId;
+    const [autoId] = useState(() => `${idPrefix}_${++idCount}`);
+    const id = externalId || (children ? autoId : undefined);
 
     return (
       <FieldLabelWrapper id={id}>