Ishmael Smyrnow

Results 14 comments of Ishmael Smyrnow

@kingrollo Those lines were likely copied verbatim from the Leaflet version of the layers class. It's possible the leaflet implementation changed as well. I'd be happy to merge a PR...

I'm not planning to add this feature myself. If someone wanted to fork and add this functionality, I'd consider merging it. It would need to be opt-in functionality (via advanced...

Can you put a newline [between these two variables](https://github.com/calvinmetcalf/Leaflet.groupedlayercontrol/commit/6a1263ce1e48bacd5d3ccb1f9ab955ef81ded5d2#diff-7a4578c3ea4c0f421057ead703fa0e5aR209), to match the other variable declarations around it? Otherwise, this looks good.

You're totally right about that. Would someone mind opening a pull request with the fix?

That sounds reasonable. I'm surprised the group checkboxes don't behave like this already. If you or another community member can open a pull request with this enhancement, I will merge...

Do you know how the standard leaflet layer control works in this regard? Most of the code for this control comes from that implementation, and we only add the extra...

You might be able to use HTML for the layer name, and include the image that way. You'll need to include some CSS to style it properly. Alternatively, someone in...

Don't worry about the branch conflicts. I had added a version to the package.json and tagged it. I can handle merging your PR.

This was a side project, and not something I actively maintain. If you want to contribute a fix for the problem in the form a pull request, that would be...