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

Use ResizeObserver to call L.Popup's update() #8223

Open
IvanSanchez opened this issue May 9, 2022 · 0 comments
Open

Use ResizeObserver to call L.Popup's update() #8223

IvanSanchez opened this issue May 9, 2022 · 0 comments

Comments

@IvanSanchez
Copy link
Member

Feature request, very similar to #8222 , but for popups instead of for the whole map.

The idea consists in attaching a ResizeObserver to the DOM element with the popup's (or the tooltip's) contents. Whenever the contents change in such a way that the container box is resized, call the update() method.

This needs a bit of research, in regards of where this functionality should go. Ideally, this would be implemented in DivOverlay so it works on both popups and tooltips.

Kinda a follow-up to #8217 . I expect there will be more than one framework, or web component, or whatever, capable of modifying the contents of a popup/tooltip without explicitly calling update().

Exactly as in #8222, this won't work on any version of IE, nor Edge < 79 (that includes all the Edge versions previous to the switch to chrome's blink engine).

@IvanSanchez IvanSanchez added feature ie Internet Explorer labels May 9, 2022
@IvanSanchez IvanSanchez added this to the 2.0 milestone May 9, 2022
@jonkoops jonkoops modified the milestones: 2.0, 3.0 May 23, 2022
@Malvoz Malvoz removed the ie Internet Explorer label May 23, 2022
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