Skip to content

Add a float image plugin#537

Merged
ocefpaf merged 2 commits intopython-visualization:masterfrom
juoceano:float_image
Oct 22, 2016
Merged

Add a float image plugin#537
ocefpaf merged 2 commits intopython-visualization:masterfrom
juoceano:float_image

Conversation

@juoceano
Copy link
Copy Markdown
Contributor

Implements #319 with a logic similar to ScrollZoomToggler instead of using L.DomUtil. It is simpler, but the relative position is not against the actual map and may be misleading.

@ocefpaf
Copy link
Copy Markdown
Member

ocefpaf commented Oct 21, 2016

Nice one!

Implements #319 with a logic similar to ScrollZoomToggler instead of using L.DomUtil. It is simpler, but the relative position is not against the actual map and may be misleading.

That is fine for now. At least people will be able to add extra image elements easier now.

Can you add a test for the new feature?

# Verify that the div has been created.
tmpl = Template("""
<img id="{{this.get_name()}}" alt="float_image"
src="https://raw.githubusercontent.com/SECOORA/static_assets/master/maps/img/rose.png"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably copy that image here.

@ocefpaf ocefpaf merged commit 59db576 into python-visualization:master Oct 22, 2016
@ocefpaf
Copy link
Copy Markdown
Member

ocefpaf commented Oct 22, 2016

Awesome! Another one bites the dust!

sanga pushed a commit to sanga/folium that referenced this pull request Oct 24, 2017
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

Successfully merging this pull request may close these issues.

2 participants