Skip to content

Commit fbf232a

Browse files
author
Ola Holmström
committed
Merge pull request #3 from tremby/patch-1
Use body as root scrollable object instead of window
2 parents eab407a + 32bdbd9 commit fbf232a

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,7 @@ npm install scrollparent --save
1414
## Example
1515

1616
```js
17-
Scrollparent(document.getElementById("content")) // window
17+
Scrollparent(document.getElementById("content")) // document.body
1818
```
1919

2020
```js

scrollparent.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
}
3131
}
3232

33-
return window;
33+
return document.body;
3434
};
3535

3636
// If common js is defined use it.

test.html

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,8 @@
127127

128128
equal(Scrollparent(document.getElementById("test1")).className, "scroll");
129129

130-
equal(Scrollparent(document.getElementById("test2")), window);
131-
equal(Scrollparent(document.getElementById("test3")), window);
130+
equal(Scrollparent(document.getElementById("test2")), document.body);
131+
equal(Scrollparent(document.getElementById("test3")), document.body);
132132

133133
equal(Scrollparent(document.getElementById("test4")).className, "scroll-y");
134134
equal(Scrollparent(document.getElementById("test5")).className, "scroll-x");

0 commit comments

Comments
 (0)