Fix recursive path in getScrollParent

Author: Madeline Trotter

src/Lumi/Components2/ScrollObserver.js

@@ -4,8 +4,11 @@
 // the first parent with a scroll bar (including auto, which
 // on some devices hides the scroll bar until hovered).
 exports.getScrollParent = node => () => {
-  const isElement = node instanceof HTMLElement;
-  const computedStyle = isElement && window.getComputedStyle(node);
+  if (!(node instanceof HTMLElement)) {
+    return document.body;
+  }
+
+  const computedStyle = window.getComputedStyle(node);
   const overflowY = computedStyle && computedStyle.overflowY;
   const overflowX = computedStyle && computedStyle.overflowX;
   const isScrollable =
@@ -14,12 +17,10 @@ exports.getScrollParent = node => () => {
     (overflowX &&
       !(overflowX.includes("visible") || overflowX.includes("hidden")));
 
-  if (!node) {
-    return document.body;
-  } else if (isScrollable && node.scrollHeight >= node.clientHeight) {
+  if (isScrollable && node.scrollHeight >= node.clientHeight) {
     return node;
   } else {
-    return exports.getScrollParent(node.parentNode);
+    return exports.getScrollParent(node.parentNode)();
   }
 };