brunob / leaflet.fullscreen

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

Updates to mouseclick handlers #88

Closed skirridsystems closed 4 months ago

skirridsystems commented 3 years ago

Prevent click and drag on the button from dragging the map. Return focus to the map after the click is handled.

brunob commented 3 years ago

Could you provide a complete description to reproduce this bug on the demo file ?

skirridsystems commented 3 years ago

Left click and drag on the fullscreen button. Observe that the map moves. Compare with +/- buttons where click and drag does not start a map move.

Maybe we don't need the doubleclick handler. I looked at the button handlers in another plugin for inspiration, unfortunately I don't remember which one now.

BePo65 commented 11 months ago

I just tried to reproduce this effect on the demo page, but I don't see it in my test environment (win10, current firefox or chrome).

What configuration do you use?

brunob commented 11 months ago

Same here with FF 118.01 on Ubuntu 20.04, i think we can close this one.