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

[Modal]On mobile devices, body element cannot scroll if there is a nested multiple modals set to observeChanges true #6611

Open
mt-u opened this issue Oct 2, 2018 · 5 comments

Comments

@mt-u
Copy link

mt-u commented Oct 2, 2018

On mobile devices, body element cannot scroll if there is a nested multiple modals set to observeChanges true

Steps

  1. Place the parent modal.
  2. Next, place the child modal inside the parent.
  3. Execute initialization script of parent and child modal. At this time, set observeChanges of the parent modal to true.

Expected Result

When a modal is hidden, the body element should be scrollable.

Actual Result

Even when a modal is hidden, the body element can not scroll.

Version

2.4.0

Testcase

https://jsfiddle.net/hogehoge/j4ztp3eL/

@y0hami
Copy link
Member

y0hami commented Oct 2, 2018

What do you mean by you can't scroll? The jsfiddle you linked doesn't have enough content in the body to make it scroll.

@mt-u
Copy link
Author

mt-u commented Oct 2, 2018

@hammy2899
Thank you for your comment.
To be more precise, you can not "touchmove" on the card.
Please see the difference in behavior between PC and mobile device.
On the PC you can scroll the area at the bottom right, whereas you can not scroll on mobile devices.
If the number of cards is not enough for your monitor to require scrolling, try adding cards in the upper left area.

@y0hami
Copy link
Member

y0hami commented Oct 2, 2018

@mt-u thank you for the clarification.

I have tested this on BrowserStack and I can confirm this is an issue.

@mt-u
Copy link
Author

mt-u commented Oct 2, 2018

@hammy2899 Cool! Thank you!

@lubber-de

This comment was marked as spam.

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

No branches or pull requests

3 participants