-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Plugins ideas for a sprint #735
Comments
Just looked at https://github.com/albburtsev/Leaflet.geojsonCSS - looks similar to what I have just submitted for a pull request but in the TimestampedGeoJson Plugin. Happy to implement this wider using their layout |
@ghandic That would be nice! Do you want to submit in the same PR or do you want to close that and send a new one? |
I’ll do it in another pull request, also looking into the leaflet-search, I’m trying to maintain short branches so will do these in separate PR’s
…Sent from my iPhone
On 13 Oct 2017, at 17:49, Filipe <notifications@github.com<mailto:notifications@github.com>> wrote:
@ghandic<https://github.com/ghandic> That would be nice! Do you want to submit in the same PR or do you want to close that and send a new one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#735 (comment)>, or mute the thread<https://github.com/notifications/unsubscribe-auth/AWaWQSjXOJt2dRufQwijlkdrfjwjVQqCks5sr5SdgaJpZM4PkL9G>.
|
Awesome! |
I'm going to look into https://github.com/albburtsev/Leaflet.geojsonCSS this week, should this be implemented as standard rather than a plugin? I have implemented this in 'features.py' could deprecate the old GeoJson in favour of the GeoJsonCss? Both are in there for now |
Thinking of incorporating BeafutifyMarker, ExtraMarkers into the Marker class rather than having two new classes. I feel like I'm putting in too many PR! Haha, what do you think @ocefpaf |
There is no such a thing as too many PRs ;-p
I'm away from office though and I'll only be able to review than next week.
|
What is needed to integrate plugins into folium? I wish it could work with arbitrary plugins (something like Plunker etc) |
Closing this for now. The list of plugin is still interesting but many of those can be implemented as external plugins. |
The text was updated successfully, but these errors were encountered: