From 0d92dcd793aea9a20897b889cc5e6ea737b41272 Mon Sep 17 00:00:00 2001 From: Armagan Ersoz Date: Tue, 8 Nov 2022 13:26:31 +1000 Subject: [PATCH 1/6] aria-hidden for visible More menu text and sr-only equavalent for screen readers and add list for tul --- src/UnderlineNav2/UnderlineNav.tsx | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/UnderlineNav2/UnderlineNav.tsx b/src/UnderlineNav2/UnderlineNav.tsx index b58470ecdc3..6d9fa255b0c 100644 --- a/src/UnderlineNav2/UnderlineNav.tsx +++ b/src/UnderlineNav2/UnderlineNav.tsx @@ -312,7 +312,7 @@ export const UnderlineNav = forwardRef( aria-label={ariaLabel} ref={navRef} > - + {responsiveProps.items} {actions.length > 0 && ( @@ -325,7 +325,10 @@ export const UnderlineNav = forwardRef( onClick={onAnchorClick} trailingIcon={TriangleDownIcon} > - More + + {`More: ${ariaLabel} items`} Date: Tue, 8 Nov 2022 13:31:21 +1000 Subject: [PATCH 2/6] add changeset --- .changeset/great-shirts-sort.md | 5 +++++ 1 file changed, 5 insertions(+) create mode 100644 .changeset/great-shirts-sort.md diff --git a/.changeset/great-shirts-sort.md b/.changeset/great-shirts-sort.md new file mode 100644 index 00000000000..aec40b9a858 --- /dev/null +++ b/.changeset/great-shirts-sort.md @@ -0,0 +1,5 @@ +--- +'@primer/react': patch +--- + +UnderlineNav2: Add aria-hidden and sr-only class support for a descriptive "More" button label From b81f66c62c2a8f7eed9ddb554aeb5d3fb0e542d0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Arma=C4=9Fan?= Date: Wed, 9 Nov 2022 08:39:51 +1000 Subject: [PATCH 3/6] Make sure to have "More" text command between visible and accessible button label Co-authored-by: Josh Black --- src/UnderlineNav2/UnderlineNav.tsx | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/UnderlineNav2/UnderlineNav.tsx b/src/UnderlineNav2/UnderlineNav.tsx index 6d9fa255b0c..b8a9665aa0f 100644 --- a/src/UnderlineNav2/UnderlineNav.tsx +++ b/src/UnderlineNav2/UnderlineNav.tsx @@ -325,10 +325,10 @@ export const UnderlineNav = forwardRef( onClick={onAnchorClick} trailingIcon={TriangleDownIcon} > -