Merge pull request #5751 from ilzamcmed/LPS-205552

fix(@clayui/picker): Update Picker markup with aria-controls
liferay · Jan 17, 2024 · 1a0c2b9 · 1a0c2b9
2 parents 51a922b + c6e38b1
commit 1a0c2b9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/packages/clay-core/src/picker/Picker.tsx b/packages/clay-core/src/picker/Picker.tsx
@@ -370,9 +370,7 @@ export function Picker<T extends Record<string, any> | string | number>({
 			<As
 				{...otherProps}
 				aria-activedescendant={active ? String(activeDescendant) : ''}
-				aria-controls={
-					active && isAppleDevice() ? ariaControls : undefined
-				}
+				aria-controls={active ? ariaControls : undefined}
 				aria-expanded={active}
 				aria-haspopup="listbox"
 				aria-owns={

diff --git a/packages/clay-core/src/picker/__tests__/__snapshots__/BasicRendering.tsx.snap b/packages/clay-core/src/picker/__tests__/__snapshots__/BasicRendering.tsx.snap
@@ -102,6 +102,7 @@ exports[`Picker basic rendering renders open component by default 1`] = `
   <div>
     <button
       aria-activedescendant="Apple"
+      aria-controls="clay-id-12"
       aria-expanded="true"
       aria-haspopup="listbox"
       aria-owns="clay-id-12"