search

brunob / leaflet.fullscreen

Leaflet.Control.FullScreen for Leaflet
https://brunob.github.io/leaflet.fullscreen/
MIT License
380 stars 110 forks source link

fix: prevent firing `exitFullscreen` event on all maps of the page in… #118

Closed brunob closed 10 months ago

brunob commented 10 months ago

… _handleFullscreenChange()

Fix #117

brunob commented 10 months ago

Any thought @BePo65 ?

brunob commented 10 months ago

We should also put map.invalidateSize(); in if (ev.target === map.getContainer() test for sanity.

BePo65 commented 10 months ago

The solution is pretty simple. but give me a few days to reproduce the effect and the solution. I think that I can give you an evaluation until this weekend.

BePo65 commented 10 months ago

So now I created a demo (on my pc only :-) and can reproduce the error and the solution. IMHO this is a fine solution that should be added to this project. 👍