fix: add overflow to body instead of html

wswebcreation · May 26, 2019 · 5991a88 · 5991a88
1 parent 9dbe93b
commit 5991a88
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/clientSideScripts/hideScrollbars.spec.ts b/lib/clientSideScripts/hideScrollbars.spec.ts
@@ -2,14 +2,14 @@ import hideScrollBars from './hideScrollbars';
 
 describe('hideScrollBars', ()=>{
   it('should be able to hide and show the scrollbars', ()=>{
-    expect(document.documentElement.style.overflow).toMatchSnapshot();
+    expect(document.body.style.overflow).toMatchSnapshot();
 
     hideScrollBars(true);
 
-    expect(document.documentElement.style.overflow).toMatchSnapshot();
+    expect(document.body.style.overflow).toMatchSnapshot();
 
     hideScrollBars(false);
 
-    expect(document.documentElement.style.overflow).toMatchSnapshot();
+    expect(document.body.style.overflow).toMatchSnapshot();
   });
 });
diff --git a/lib/clientSideScripts/hideScrollbars.ts b/lib/clientSideScripts/hideScrollbars.ts
@@ -3,8 +3,8 @@
  */
 export default function hideScrollBars(hide: boolean): void {
   if (hide) {
-    document.documentElement.style.overflow = 'hidden';
+    document.body.style.overflow = 'hidden';
   } else {
-    document.documentElement.style.overflow = '';
+    document.body.style.overflow = '';
   }
 }