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

[OverlayManager] scroll lock classes not removed from body in iOS 15 #2488

Open
Hzunax opened this issue Mar 6, 2025 · 0 comments
Open

[OverlayManager] scroll lock classes not removed from body in iOS 15 #2488

Hzunax opened this issue Mar 6, 2025 · 0 comments

Comments

@Hzunax
Copy link
Contributor

Hzunax commented Mar 6, 2025

Expected behavior

When an overlay is removed from the DOM, the scroll lock classes should be removed from the <body>

Actual Behavior

Works in Web, Android and iOS 16+. But in iOS 15, the scroll lock classes remain present on the <body>, therefore preventing scrolling of the content below.

The overlay is not closed with a close-overlay event, but rather removed from DOM, as you can see in the Stackblitz example.

In iOS 18, the OverlayManager's hide() is called before the remove().
In iOS 15, it is the other way around, first remove(), then hide(), resulting in the following error in the console: 'could not find controller to hide'.

https://studio.webcomponents.dev/edit/HnMxOof4alHm2BV3Q0EJ/README.md?p=stories

Additional context

lion-dialog

Screenshot in Simulator with iOS 18:

Image

Screenshot in Simulator with iOS 15, showing error:

Image

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

No branches or pull requests

1 participant