Skip to content

Commit 6300208

Browse files
committed
fixes iliakan#34
1 parent e5fca50 commit 6300208

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

  • 2-ui/1-document/16-metrics-window/1-pageyoffset-polyfill

2-ui/1-document/16-metrics-window/1-pageyoffset-polyfill/solution.md

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,9 @@ window.pageYOffset = document.documentElement.scrollTop;
1515
alert( window.pageYOffset );
1616
```
1717

18-
Код выше не учитывает текущую прокрутку. Он присваивает `window.pageYOffset` один раз и в дальнейшем, чтобы получить текущую прокрутку, нужно снова обратиться к `document.documentElement.scrollTop` не меняет его. А задача как раз -- сделать полифилл, то есть дать возможность использовать `window.pageYOffset` для получения текущего состояния прокрутки без "танцев бубном", так же как в современных браузерах.
18+
Код выше не учитывает текущую прокрутку. Он присваивает `window.pageYOffset` текущую прокрутку, но при её изменении -- не обновляет это свойство автоматически, а поэтому -- бесполезен.
1919

20-
Для этого создадим свойство через геттер.
21-
22-
В IE8 для DOM-объектов работает `Object.defineProperty`:
20+
Более правильное решение -- сделать это свойство геттером. При этом в IE8 для DOM-объектов работает `Object.defineProperty`:
2321

2422
```js
2523
//+ run

0 commit comments

Comments
 (0)