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

Rewrote the plugin #17

Open
bhaskarvk opened this issue Nov 13, 2016 · 4 comments
Open

Rewrote the plugin #17

bhaskarvk opened this issue Nov 13, 2016 · 4 comments

Comments

@bhaskarvk
Copy link

I'm not creating a PR but I would like you to take a look at my fork

https://github.com/bhaskarvk/leaflet-choropleth

Here's what I've done to it

  • Added support for more chroma-js options like bezier interpolate, fix lightness, channel mode
  • Rewrote it so that it can work along with omnivore. For now though it requires my fork of omnivore. This allows the plugin to be used not only for GeoJSON but every file that omnivore supports so TopoJSON/KML/GFX/CSV/WKT
  • Added ability to customize and add a legend automatically if so desired.

Please let me know what you think, and whether you would be interested in merging this.

@abrin
Copy link
Contributor

abrin commented Nov 13, 2016

Hi, it seems like you did a ton of work, but, I can’t seem to find documentation of how to use the features, examples, or tests for those features, could you point me to them?

Thanks

On Nov 12, 2016, at 5:06 PM, Bhaskar V. Karambelkar notifications@github.com wrote:

I'm not creating a PR but I would like you to take a look at my fork

https://github.com/bhaskarvk/leaflet-choropleth https://github.com/bhaskarvk/leaflet-choropleth
Here's what I've done to it

Added support for more chroma-js options like bezier interpolate, fix lightness, channel mode
Rewrote it so that it can work along with omnivore x-msg://3/mapbox/leaflet-omnivore. For now though it requires my fork of omnivore x-msg://3/bhaskarvk/leaflet-omnivore. This allows the plugin to be used not only for GeoJSON but every file that omnivore supports so TopoJSON/KML/GFX/CSV/WKT
Added ability to customize and add a legend automatically if so desired.
Please let me know what you think, and whether you would be interested in merging this.


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub #17, or mute the thread https://github.com/notifications/unsubscribe-auth/AALwrmAVGvRwRtl9MX0dMDjDgqxG3T1dks5q9lSMgaJpZM4KwkIq.

@bhaskarvk
Copy link
Author

Hi,
I haven't got to that part yet. I did test it out locally, I'll update the examples soon. Thanks!

@abrin
Copy link
Contributor

abrin commented Nov 13, 2016

Cool. I didn't mean to question the work that you did, just without the former, it's really hard to get a handle around the improvements and changes.

@bhaskarvk
Copy link
Author

I got caught in some other stuff. I'll revisit this and add proper examples. Thanks for your patience.

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