Skip to content
This repository has been archived by the owner on Nov 27, 2018. It is now read-only.

Fix for hidden overflow #113

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix for hidden overflow #113

wants to merge 2 commits into from

Conversation

k-one-o-two
Copy link

In case of hidden overflow: IE should not include margin (added for the native scrollbar, which is hidden) in scrollTop

@k-one-o-two
Copy link
Author

This fixes #112

@k-one-o-two
Copy link
Author

Found a better solution: see my commit

@gromo
Copy link
Owner

gromo commented Nov 23, 2016

I'm still thinking it's better to update browser scroll calculation function

@k-one-o-two
Copy link
Author

But it affects the behavior of all scrolls on the page which is not good. My change really checks if this is needed

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants