refactor(tabs): simplify looping through tabs logic

elastic · Nov 21, 2024 · 6a5fa56 · 6a5fa56
1 parent 2075b01
commit 6a5fa56
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/eui/src/components/tabs/tabs.tsx b/packages/eui/src/components/tabs/tabs.tsx
@@ -80,10 +80,12 @@ export const EuiTabs = forwardRef<EuiTabRef, EuiTabsProps>(
       );
 
       if (event.key === keys.ARROW_LEFT) {
-        const previousIndex = (currentIndex - 1 + tabs.length) % tabs.length;
+        const previousIndex =
+          (currentIndex === 0 ? tabs.length : currentIndex) - 1;
         tabs[previousIndex].focus();
       } else if (event.key === keys.ARROW_RIGHT) {
-        const nextIndex = (currentIndex + 1) % tabs.length;
+        const nextIndex =
+          currentIndex === tabs.length - 1 ? 0 : currentIndex + 1;
         tabs[nextIndex].focus();
       }
     };