fix(utils): Improve getScrollParent

src/utils/getScrollParent.js

@@ -0,0 +1,21 @@
+function getScrollParent(node) {
+	const isElement = node instanceof HTMLElement;
+	const overflowY = isElement && window.getComputedStyle(node).overflowY;
+	const isScrollable = !(
+		overflowY.includes('hidden') || overflowY.includes('visible')
+	);
+
+	if (!node) {
+		return null;
+	} else if (isScrollable && node.scrollHeight >= node.clientHeight) {
+		return node;
+	}
+
+	return (
+		getScrollParent(node.parentNode) ||
+		document.scrollingElement ||
+		document.body
+	);
+}
+
+export default getScrollParent;

src/utils/index.js

@@ -2,11 +2,9 @@ export * from './colors';
 export {default as mergeRefs} from './mergeRefs';
 export {default as mergeCallbacks} from './mergeCallbacks';
 export {default as getLength} from './getLength';
+export {default as getScrollParent} from './getScrollParent';
 export {default as getSpacing} from './getSpacing';
-export {
-	default as scrollIntoViewIfNeeded,
-	getScrollParent,
-} from './scrollIntoViewIfNeeded';
+export {default as scrollIntoViewIfNeeded} from './scrollIntoViewIfNeeded';
 export * from './spacing';
 export {default as truncateText} from './truncateText';
 export * from './units';

src/utils/scrollIntoViewIfNeeded.js

@@ -1,13 +1,4 @@
-function getScrollParent(element) {
-	if (!element) {
-		return null;
-	}
-	if (element.scrollHeight > element.clientHeight) {
-		return element;
-	} else {
-		return getScrollParent(element.parentNode);
-	}
-}
+import getScrollParent from './getScrollParent';
 
 /**
  * Vertically Scrolls an element into view if it's not visible yet.
@@ -32,6 +23,4 @@ function scrollIntoViewIfNeeded(element) {
 	}
 }
 
-export {getScrollParent};
-
 export default scrollIntoViewIfNeeded;