fix: Tabs should activate tab when press enter/space on unactive tab (#825) (#826)

Author: afc163

docs/examples/basic.tsx

@@ -41,16 +41,16 @@ export default () => {
     console.log('onTabClick', key);
   };
 
-  const onChange = (key: string) => {
-    console.log('onChange', key);
+  const onTabChange = (key: string) => {
+    console.log('onTabChange', key);
   };
 
   return (
     <React.StrictMode>
       <Tabs
         tabBarExtraContent="extra"
         onTabClick={onTabClick}
-        onChange={onChange}
+        onChange={onTabChange}
         direction={direction}
         items={items}
       />

src/TabNavList/index.tsx

@@ -375,7 +375,7 @@ const TabNavList = React.forwardRef<HTMLDivElement, TabNavListProps>((props, ref
       case 'Enter':
       case 'Space': {
         e.preventDefault();
-        onTabClick(activeKey, e);
+        onTabClick(focusKey ?? activeKey, e);
         break;
       }
       // Backspace

tests/accessibility.test.tsx

@@ -102,15 +102,24 @@ describe('Tabs.Accessibility', () => {
     // activate tab
     await user.keyboard(' ');
     expect(onTabClick).toHaveBeenCalledTimes(1);
-    expect(onChange).not.toHaveBeenCalled();
+    expect(onTabClick).toHaveBeenLastCalledWith('1', expect.any(Object));
+    expect(onChange).toHaveBeenCalledTimes(0);
 
     // move focus to second tab
     await user.keyboard('{ArrowRight}');
 
     // activate tab
     await user.keyboard('{Enter}');
     expect(onTabClick).toHaveBeenCalledTimes(2);
-    expect(onChange).not.toHaveBeenCalled();
+    expect(onTabClick).toHaveBeenLastCalledWith('2', expect.any(Object));
+    expect(onChange).toHaveBeenCalledTimes(1);
+    expect(onChange).toHaveBeenLastCalledWith('2');
+
+    // press enter on same tab
+    await user.keyboard('{Enter}');
+    expect(onTabClick).toHaveBeenCalledTimes(3);
+    expect(onTabClick).toHaveBeenLastCalledWith('2', expect.any(Object));
+    expect(onChange).toHaveBeenCalledTimes(1);
   });
 
   it('should not navigate to disabled tabs', async () => {