From 02087312db613e425b7e30ffc5d14c0fcc8a38f6 Mon Sep 17 00:00:00 2001
From: Adam Giebl <adamgiebl@gmail.com>
Date: Wed, 13 Dec 2023 21:30:24 +0100
Subject: [PATCH 1/2] renaming useIsScrolled

---
 packages/jui/src/Tabs/Tabs.tsx                 |  8 ++++----
 .../{useIsScrolled.tsx => useHasOverflow.tsx}  | 18 +++++++++---------
 2 files changed, 13 insertions(+), 13 deletions(-)
 rename packages/jui/src/Tabs/{useIsScrolled.tsx => useHasOverflow.tsx} (69%)

diff --git a/packages/jui/src/Tabs/Tabs.tsx b/packages/jui/src/Tabs/Tabs.tsx
index a9d0860a..8d7e40fd 100644
--- a/packages/jui/src/Tabs/Tabs.tsx
+++ b/packages/jui/src/Tabs/Tabs.tsx
@@ -6,7 +6,7 @@ import { AriaTabListProps } from "@react-types/tabs";
 import { StyledHorizontalOverflowShadows } from "./StyledHorizontalOverflowShadows";
 import { TabsOverflowMenu } from "./TabsOverflowMenu";
 import { useCollectionOverflowObserver } from "./useCollectionOverflowObserver";
-import { useIsScrolled } from "./useIsScrolled";
+import { useHasOverflow } from "./useHasOverflow";
 import { styled, css } from "@intellij-platform/core/styled";
 import { StyledDefaultTab } from "./StyledDefaultTab";
 import { StyledDefaultTabs } from "./StyledDefaultTabs";
@@ -119,7 +119,7 @@ export const Tabs = <T extends object>({
   const ref = React.useRef<HTMLDivElement>(null);
   const { tabListProps } = useTabList(props, state, ref);
 
-  const { scrolledIndicatorProps, isScrolled } = useIsScrolled({ ref });
+  const { scrolledIndicatorProps, hasOverflow } = useHasOverflow({ ref });
   const { overflowedKeys, intersectionObserver } =
     useCollectionOverflowObserver(ref);
 
@@ -145,8 +145,8 @@ export const Tabs = <T extends object>({
   return (
     <TabsComponent noBorders={noBorders} {...filterDOMProps(props)}>
       <StyledHorizontalOverflowShadows
-        hasOverflowAtStart={isScrolled.left}
-        hasOverflowAtEnd={isScrolled.right}
+        hasOverflowAtStart={hasOverflow.left}
+        hasOverflowAtEnd={hasOverflow.right}
         style={{ minWidth: 0 }}
       >
         <StyledTabList
diff --git a/packages/jui/src/Tabs/useIsScrolled.tsx b/packages/jui/src/Tabs/useHasOverflow.tsx
similarity index 69%
rename from packages/jui/src/Tabs/useIsScrolled.tsx
rename to packages/jui/src/Tabs/useHasOverflow.tsx
index 74a9d829..9ffa8925 100644
--- a/packages/jui/src/Tabs/useIsScrolled.tsx
+++ b/packages/jui/src/Tabs/useHasOverflow.tsx
@@ -1,13 +1,13 @@
 import { RefObject, UIEventHandler, useEffect, useState } from "react";
 
-export function useIsScrolled<T extends HTMLElement>({
+export function useHasOverflow<T extends HTMLElement>({
   threshold = 5,
   ref,
 }: {
   threshold?: number;
   ref: RefObject<T>;
 }) {
-  const [isScrolled, setIsScrolled] = useState({
+  const [hasOverflow, setHasOverflow] = useState({
     left: false,
     right: false,
     top: false,
@@ -23,19 +23,19 @@ export function useIsScrolled<T extends HTMLElement>({
       const offsetTop = element.scrollTop;
       const offsetBottom =
         element.scrollHeight - (element.offsetHeight + element.scrollTop);
-      const newIsScrolled = {
+      const newHasOverflow = {
         top: offsetTop >= threshold,
         bottom: offsetBottom >= threshold,
         left: offsetLeft >= threshold,
         right: offsetRight >= threshold,
       };
       if (
-        isScrolled.top !== isScrolled.top ||
-        isScrolled.bottom !== newIsScrolled.bottom ||
-        isScrolled.left !== newIsScrolled.left ||
-        isScrolled.right !== newIsScrolled.right
+        hasOverflow.top !== newHasOverflow.top ||
+        hasOverflow.bottom !== newHasOverflow.bottom ||
+        hasOverflow.left !== newHasOverflow.left ||
+        hasOverflow.right !== newHasOverflow.right
       ) {
-        setIsScrolled(newIsScrolled);
+        setHasOverflow(newHasOverflow);
       }
     }
   };
@@ -45,6 +45,6 @@ export function useIsScrolled<T extends HTMLElement>({
     scrolledIndicatorProps: {
       onScroll: update as UIEventHandler<T>,
     },
-    isScrolled,
+    hasOverflow,
   };
 }

From 53f30e542f9bba03be4e70179ac00106a09b0492 Mon Sep 17 00:00:00 2001
From: Adam Giebl <adamgiebl@gmail.com>
Date: Wed, 13 Dec 2023 21:32:52 +0100
Subject: [PATCH 2/2] adjusting comparator for threshold 0

---
 packages/jui/src/Tabs/useHasOverflow.tsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/jui/src/Tabs/useHasOverflow.tsx b/packages/jui/src/Tabs/useHasOverflow.tsx
index 9ffa8925..03573eb9 100644
--- a/packages/jui/src/Tabs/useHasOverflow.tsx
+++ b/packages/jui/src/Tabs/useHasOverflow.tsx
@@ -24,10 +24,10 @@ export function useHasOverflow<T extends HTMLElement>({
       const offsetBottom =
         element.scrollHeight - (element.offsetHeight + element.scrollTop);
       const newHasOverflow = {
-        top: offsetTop >= threshold,
-        bottom: offsetBottom >= threshold,
-        left: offsetLeft >= threshold,
-        right: offsetRight >= threshold,
+        top: offsetTop > threshold,
+        bottom: offsetBottom > threshold,
+        left: offsetLeft > threshold,
+        right: offsetRight > threshold,
       };
       if (
         hasOverflow.top !== newHasOverflow.top ||