jjimenezshaw / Leaflet.Control.Layers.Tree

a Tree Layers Control for Leaflet
https://jjimenezshaw.github.io/Leaflet.Control.Layers.Tree/examples/basic.html
BSD 3-Clause "New" or "Revised" License
146 stars 36 forks source link

Prevent click propagation and collapsing tree on mobile browsers #70

Closed Falke-Design closed 11 months ago

Falke-Design commented 1 year ago

Uses the same solution as Leaflet to fix click propagation on mbile browsers. https://github.com/Leaflet/Leaflet/issues/8778, https://github.com/Leaflet/Leaflet/pull/8910

Fixes: #67

@PedroVenancio, @jjimenezshaw

PedroVenancio commented 1 year ago

Wonderful @Falke-Design It works like a charm!!

Thank you very much!!

jjimenezshaw commented 1 year ago

I think this is merged, should this be closed?

I have not merged it. I have had no time to test it, and probably I will have not time the next couple of weeks, sorry, I hope in July I can do it.

jjimenezshaw commented 1 year ago

Hi. Finally I have been testing it with examples/basic.html (updating to 1.9.4) and it works if I click on the radio button. However it does not work properly if I click on the label next to the first radio button, OSM. It collapses the group. In the label Open TopoMap it seems to work properly. Do you know why?

jjimenezshaw commented 11 months ago

Sorry for the late response. I thought that a strange behaviour was due to these changes, but I confirmed that it was there before. Do you need a release for that?