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

fix: scrolling page in the background on iOS (#1956) #2521

Closed

Conversation

goestav
Copy link

@goestav goestav commented Oct 11, 2022

This pull request fixes a bug where you could scroll the background (page) when you reached the end of the alert.
Closes #1956

This commit fixes a bug where you could scroll the background (page) when you reached the end of the alert.
@limonte
Copy link
Member

limonte commented Oct 11, 2022

Thank you for the contribution @goestav

Unfortunately, the issue isn't fixed by adding this CSS rule. The preview is deployed here: https://deploy-preview-2521--sweetalert2.netlify.app/

Steps to reproduce the issue:

  1. open https://deploy-preview-2521--sweetalert2.netlify.app/
  2. click the "Try me" button related to the "A modal window with a long content inside:" example
  3. RIGHT AFTER clicking the "Try me" button scroll down

@goestav
Copy link
Author

goestav commented Oct 11, 2022

hmm strange, I can reproduce it on the preview but not locally

@goestav
Copy link
Author

goestav commented Oct 14, 2022

I'm pretty sure this is a webkit bug like brunoinds mentioned and not related to sweetalert2.
The only solution I managed to get to work was the one from Rik Schennink but it's hacky and will probably mess up the layout of some projects because of the position: fixed; property.

@goestav goestav closed this Oct 14, 2022
@limonte
Copy link
Member

limonte commented Oct 15, 2022

Thank you for the investigation and the feedback @goestav!

@goestav goestav deleted the fix/ios-background-scroll-bug branch October 15, 2022 23:09
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.

iOS - the page is being scrolled in background for tall modals
2 participants