refactor(tabs): simplify looping through tabs logic

weronikaolejniczak · weronikaolejniczak · commit b25165db6a32 · 2024-11-20T14:49:48.000+01:00
diff --git a/packages/eui/src/components/tabs/tabs.tsx b/packages/eui/src/components/tabs/tabs.tsx
@@ -80,10 +80,11 @@ 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 ? 0 : currentIndex + 1;
         tabs[nextIndex].focus();
       }
     };
