fix (a11y): set aria-required to the correct value

satellytes · Sep 19, 2024 · 453e55d · 453e55d
1 parent 6d9e5b6
commit 453e55d
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/components/forms/checkbox/checkbox.tsx b/src/components/forms/checkbox/checkbox.tsx
@@ -84,7 +84,7 @@ export const Checkbox = (
     <div>
       <CheckboxLabel>
         <HiddenCheckbox
-          aria-required={true}
+          aria-required={Boolean(props.rules?.required)}
           aria-describedby={errorMessage && `${props.name}-error`}
           aria-labelledby={props.label && `${props.name}-label`}
           aria-invalid={Boolean(errorMessage)}

diff --git a/src/components/forms/text-area/text-area.tsx b/src/components/forms/text-area/text-area.tsx
@@ -46,7 +46,7 @@ export const TextArea = (
         </Label>
       )}
       <StyledTextArea
-        aria-required={true}
+        aria-required={Boolean(props.rules?.required)}
         aria-describedby={errorMessage && `${props.name}-error`}
         aria-invalid={Boolean(errorMessage)}
         $hasError={Boolean(errorMessage)}

diff --git a/src/components/forms/text-input/text-input.tsx b/src/components/forms/text-input/text-input.tsx
@@ -65,7 +65,7 @@ export const TextInput = (
       )}
       <Input
         type={'text'}
-        aria-required={true}
+        aria-required={Boolean(props.rules?.required)}
         aria-describedby={errorMessage && `${props.name}-error`}
         aria-invalid={Boolean(errorMessage)}
         $hasError={Boolean(errorMessage)}