-
Notifications
You must be signed in to change notification settings - Fork 889
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
v1 todo #87
Comments
Hi Paul, I like the potential of this library for quickly bootstrapping geospatial apps written in React. Are there are any tasks you can use help with? |
Hi @jonboiser, thanks for asking! I think adding the attribution, layers and scale controls is the next priority. It shouldn't be a lot of work, just make sure they behave as expected. Leaflet v1 support is tricky because I want to have it as soon as it's stable, but the current betas seem to change the APIs so I'm not sure it's worth investing time on it. |
Cool. I'll take a look at how you've implemented the Zoom control. I think I can help with the docs and website, once I play around with the library. At my company, we started a similar project for the ArcGIS JS API and Leaflet, but think we should deprecate the Leaflet piece, since this library is available. |
I pushed some code for the AttributionControl to my fork. It seems like coding the other controls will be pretty much the same. I'll tackle the others during the week and submit a PR. |
What's the progress like on this? |
Mostly waiting on a RC of Leaflet v1 to start adapting what needs to be, and the final v1 release to effectively version this lib as v1 as well. |
Any news on this, since meanwhile there is even a second RC available? |
Closing this issue, please go to the v1 release plan for more information about this future release. |
Must have
CONTRIBUTING.md
ISSUE_TEMPLATE.md
onLeaflet...
eventsgetLeafletElement()
methodNice to have
The text was updated successfully, but these errors were encountered: