`removeLayer()` not working properly from within `map.on` about leaflet-panel-layers

[designgears]

After some more digging I found that the order things are added and removed matter when inside map.on. (for reasons unknown to me)

https://github.com/stefanocudini/leaflet-panel-layers/blob/99f0ee2959454afbcbc5ec15b9804ea506519c31/src/leaflet-panel-layers.js#LL422C2-L431C2

You're looping over elements in the panel, while going down the list, the thing that comes first gets removed and the thing you clicked gets added. Going up the list, you add before you remove.

I fixed this by adding two vars, toBeAdded and toBeRemoved, replaced addLayer and removeLayer in the if/else and stored the object in those, then simply fired removeLayer(toBeRemoved) followed by addLayer(toBeAdded) after the if/else to guarantee we always do the remove before the add.