leaflet-groupedlayercontrol
leaflet-groupedlayercontrol copied to clipboard
fix for removeLayer
Isolates the fix in https://github.com/ismyrnow/leaflet-groupedlayercontrol/pull/46 for removeLayer() - per comments in that pull request, the fix in https://github.com/ismyrnow/leaflet-groupedlayercontrol/pull/50 wasn't complete and we were still seeing the removeLayer error https://github.com/ismyrnow/leaflet-groupedlayercontrol/issues/49 . Have tested this and it solves the issue (thanks @ForgottenLords )
Can you please merge this fix into whatever branch, so I can get it with NPM ? I just spend several hours pulling my hair out till I found the source of the bug and got finally to this PR 😭
OMG merge this please !
Merge this please. very basic feature that should've been present from the beginning.
Any update on merging this changeset?
@ka7eh This package is dead because layers control now included in leaflet http://leafletjs.com/reference-1.3.0.html#control-layers
@TPABHuKOB thanks for the update.
@TPABHuKOB I do not think Leaflets layer controll is a full replacement of the groupedlayercontrol. How do you do grouping - headings ? You can pass HTML in the Layers labels, but the controls - checkboxes, radios - are placed before the labels. Please see my example.
I can confirm from my local installation that this pull request fixes the problem with removeLayer
. It is a single-line fix, and definitely should be merged into master.
Also confirmed, nice fix.
Owner? I'm just another person who has faced this problem and solved it myself ...