fix: scrollable node is not scrollable in some situations (#99)

close ant-design/ant-design#6037
react-component · Aug 23, 2017 · 759f8da · 759f8da
1 parent 47b09bf
commit 759f8da
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/src/createDOMForm.js b/src/createDOMForm.js
@@ -37,7 +37,9 @@ function getScrollableContainer(n) {
   /* eslint no-cond-assign:0 */
   while ((nodeName = node.nodeName.toLowerCase()) !== 'body') {
     const overflowY = computedStyle(node, 'overflowY');
-    if (node !== n && (overflowY === 'auto' || overflowY === 'scroll')) {
+    // https://stackoverflow.com/a/36900407/3040605
+    if (node !== n && (overflowY === 'auto' || overflowY === 'scroll') &&
+        node.scrollHeight > node.clientHeight) {
       return node;
     }
     node = node.parentNode;