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

Add popup option to geo_json() #3

Closed
araichev opened this issue May 29, 2013 · 1 comment
Closed

Add popup option to geo_json() #3

araichev opened this issue May 29, 2013 · 1 comment

Comments

@araichev
Copy link

This functionality would be useful, and adding it looks doable via Leaflet's onEachFeature: http://leafletjs.com/examples/geojson.html.

@wrobstory
Copy link
Contributor

I thought this was going to be a quick fix. Turns out it's easy to add text popup content, but adding Vincent/Vega popups is a bit tricker because a number of methods and attributes aren't available on the geoJson object. I'll keep tinkering (and pull requests are welcome!)

@ocefpaf ocefpaf closed this as completed Dec 2, 2014
@ocefpaf ocefpaf reopened this Apr 9, 2015
@ocefpaf ocefpaf mentioned this issue Apr 18, 2015
@ocefpaf ocefpaf closed this as completed Apr 22, 2015
Conengmo pushed a commit to Conengmo/folium that referenced this issue Jul 20, 2018
ocefpaf pushed a commit that referenced this issue Jul 19, 2019
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

3 participants