From cab13b83d80a51dbc96bb3af79a8eddb1fc13f6c Mon Sep 17 00:00:00 2001
From: Jony J <1844749591@qq.com>
Date: Fri, 7 Feb 2025 10:32:51 +0800
Subject: [PATCH] feat: improve accessibility (#775)

* feat: improve accessibility

* fix: remove unnecessary role attributes from tab-related buttons

* Revert "fix: remove unnecessary role attributes from tab-related buttons"

This reverts commit 4c68273d6df622d1377d7baadb327986684e5959.

* revert: restore role attribute for add button
---
 src/TabNavList/OperationNode.tsx           | 2 --
 src/TabNavList/TabNode.tsx                 | 1 +
 tests/__snapshots__/index.test.tsx.snap    | 4 ----
 tests/__snapshots__/overflow.test.tsx.snap | 2 --
 4 files changed, 1 insertion(+), 8 deletions(-)

diff --git a/src/TabNavList/OperationNode.tsx b/src/TabNavList/OperationNode.tsx
index 972da8fd..87a44c80 100644
--- a/src/TabNavList/OperationNode.tsx
+++ b/src/TabNavList/OperationNode.tsx
@@ -202,8 +202,6 @@ const OperationNode = React.forwardRef<HTMLDivElement, OperationNodeProps>((prop
         type="button"
         className={`${prefixCls}-nav-more`}
         style={moreStyle}
-        tabIndex={-1}
-        aria-hidden="true"
         aria-haspopup="listbox"
         aria-controls={popupId}
         id={`${id}-more`}
diff --git a/src/TabNavList/TabNode.tsx b/src/TabNavList/TabNode.tsx
index d11c34cb..d3c0d944 100644
--- a/src/TabNavList/TabNode.tsx
+++ b/src/TabNavList/TabNode.tsx
@@ -126,6 +126,7 @@ const TabNode: React.FC<TabNodeProps> = props => {
       {removable && (
         <button
           type="button"
+          role="tab"
           aria-label={removeAriaLabel || 'remove'}
           tabIndex={active ? 0 : -1}
           className={`${tabPrefix}-remove`}
diff --git a/tests/__snapshots__/index.test.tsx.snap b/tests/__snapshots__/index.test.tsx.snap
index e742da9f..c4500f4e 100644
--- a/tests/__snapshots__/index.test.tsx.snap
+++ b/tests/__snapshots__/index.test.tsx.snap
@@ -73,11 +73,9 @@ exports[`Tabs.Basic Normal 1`] = `
         aria-controls="rc-tabs-test-more-popup"
         aria-expanded="false"
         aria-haspopup="listbox"
-        aria-hidden="true"
         class="rc-tabs-nav-more"
         id="rc-tabs-test-more"
         style="visibility: hidden; order: 1;"
-        tabindex="-1"
         type="button"
       >
         More
@@ -148,11 +146,9 @@ exports[`Tabs.Basic Skip invalidate children 1`] = `
         aria-controls="rc-tabs-test-more-popup"
         aria-expanded="false"
         aria-haspopup="listbox"
-        aria-hidden="true"
         class="rc-tabs-nav-more"
         id="rc-tabs-test-more"
         style="visibility: hidden; order: 1;"
-        tabindex="-1"
         type="button"
       >
         More
diff --git a/tests/__snapshots__/overflow.test.tsx.snap b/tests/__snapshots__/overflow.test.tsx.snap
index 759e5637..5d175809 100644
--- a/tests/__snapshots__/overflow.test.tsx.snap
+++ b/tests/__snapshots__/overflow.test.tsx.snap
@@ -5,11 +5,9 @@ exports[`Tabs.Overflow should collapse 1`] = `
   aria-controls="rc-tabs-test-more-popup"
   aria-expanded="false"
   aria-haspopup="listbox"
-  aria-hidden="true"
   class="rc-tabs-nav-more"
   id="rc-tabs-test-more"
   style=""
-  tabindex="-1"
   type="button"
 >
   More