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

Prevent scrolling behind search modal - Alternate #382

Closed
wants to merge 3 commits into from

Conversation

iandunn
Copy link
Member

@iandunn iandunn commented Mar 31, 2023

Fixes #371
Closes #381

This is an alternate approach, because applying the hotfix in #381 caused uninteded consequences. Fixing those caused more problems, which in turn had to be fixed, causing more problems.

@iandunn iandunn self-assigned this Mar 31, 2023
@iandunn
Copy link
Member Author

iandunn commented Apr 3, 2023

This is just a WIP at the moment, and needs more testing. So far though it seems more promising than #381, though.

@iandunn
Copy link
Member Author

iandunn commented Apr 25, 2023

#392 will fix this

@iandunn iandunn closed this Apr 25, 2023
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.

Search: Mobile overlay does not properly lock the screen in place
1 participant