Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

extendLayerControl should be extendLayersControl + some modifications #98

Open
tim-salabim opened this issue Nov 21, 2024 · 2 comments
Open

Comments

@tim-salabim
Copy link
Member

@trafficonese I renamed extendLayerControl() to extendLayersControl() (mind the 's') to be in line with the original leaflet::addLayersControl().
I also adjusted a few things regarding CSS and default icons. Among other things, I added package {fontawesome} to Suggests, because I like those icons much better than the emojis.

All in all it now looks like this:
image

I am still not sure whether this is the end of what I have in mind, but I wanted to let you in on these changes as early as possible, given that you did all the work on this so far.

I hope that is OK with you.
18b80d6

@trafficonese
Copy link
Contributor

looks great to me :)

@tim-salabim
Copy link
Member Author

There's still a few things I nee to iron out, but I am quite happy with this myself! Thanks again for all the great work!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants