Return scrollingElement || documentElement rather than body

tremby · tremby · commit 7138ab168133 · 2016-03-18T19:56:29.000-07:00
See https://miketaylr.com/posts/2014/11/document-body-scrolltop.html and https://groups.google.com/a/chromium.org/forum/#!msg/blink-dev/75hVThfrJ08/YD9kSgQljFEJ
diff --git a/README.md b/README.md
@@ -13,7 +13,7 @@ npm install scrollparent --save
 ```js
 var Scrollparent = require("scrollparent");
 
-Scrollparent(document.getElementById("content")) // document.body
+Scrollparent(document.getElementById("content")) // HTMLHtmlElement or HTMLBodyElement as appropriate
 ```
 
 ```js
diff --git a/scrollparent.js b/scrollparent.js
@@ -40,7 +40,7 @@
       }
     }
 
-    return document.body;
+    return document.scrollingElement || document.documentElement;
   };
 
   return scrollParent;
diff --git a/test.html b/test.html
@@ -127,8 +127,8 @@
 
 equal(Scrollparent(document.getElementById("test1")).className, "scroll");
 
-equal(Scrollparent(document.getElementById("test2")), document.body);
-equal(Scrollparent(document.getElementById("test3")), document.body);
+equal(Scrollparent(document.getElementById("test2")), document.scrollingElement || document.documentElement);
+equal(Scrollparent(document.getElementById("test3")), document.scrollingElement || document.documentElement);
 
 equal(Scrollparent(document.getElementById("test4")).className, "scroll-y");
 equal(Scrollparent(document.getElementById("test5")).className, "scroll-x");

Original file line number	Diff line number	Diff line change
`@@ -40,7 +40,7 @@`
`40`	`40`	`}`
`41`	`41`	`}`
`42`	`42`
`43`		`- return document.body;`
	`43`	`+ return document.scrollingElement \|\| document.documentElement;`
`44`	`44`	`};`
`45`	`45`
`46`	`46`	`return scrollParent;`