Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scrolling When Modals are Open #92

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Scrolling When Modals are Open #92

wants to merge 1 commit into from

Conversation

duner
Copy link
Contributor

@duner duner commented Nov 9, 2015

There might be a few performance issues that I should take a look at (or, it might just be that my computer's power is super low and I have over fifty tabs open at the moment) but this works in Chrome and Firefox.

Solves #71. I think.

@duner duner changed the title Handle Scrolling Scrolling When Modals are Open Nov 9, 2015
@duner
Copy link
Contributor Author

duner commented Nov 9, 2015

Right now this stops the scrolling in this particular instance.

@JoeGermuska, you've got a better sense of how all the components fit together — would it make sense to make this a slightly more abstract process so that other components can also prevent scrolling in their parent page-item? It could be a class that gets toggled?

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

Successfully merging this pull request may close these issues.

1 participant