Fix leaflet-layer-group #38
Conversation
In my testcase the layer-group is still not displayed on the map, but at least it does not throw errors causing the map to not render anymore...
Added missing semicolon & <content></content> so children can actually be found.
Compufreak345
changed the title
|
Hmm I just saw that for dynamically added (dom-repeat) items this only works in Chrome right now, in Firefox & Safari the elements inside the group do not get drawn if added dynamically. Static items work. I'll have a deeper look into it tomorrow. Probably has something to do with the MutationObserver. |
Needed for Firefox & Safari to work with dom-repeat.
Might be better to reregister the mutationObserver at the right point, but until now I did not see any issues.
|
I fixed the problem mentioned in my last comment by re-adding the mutation observer like mentioned in http://stackoverflow.com/questions/31156079/how-to-use-mutation-observers-in-polymer-1-0/31571790#31571790 . There also was an error loading the map in Safari caused by trying to detach a non-existing mutation observer in leaflet-core. I just checked if it exists and did not see any negative side effects, but it's probably not the most elegant solution. |
Fixed a typo in a method name, added a semicolon, added content-tag so this.children actually finds children.