Skip to content

Update TimeDimension.Control css-link on TimestampedGeoJson plugin #963

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

Merged
merged 3 commits into from
Sep 17, 2018

Conversation

frodebjerke
Copy link
Contributor

The styling of the TimeDimension Controller on the TimestampedGeoJson plugin was broken in my browser (chrome) due to the css-link being HTTP while running on an HTTPS site.

Used the content URL used in the Leaflet.TimeDimension Readme instead.

Copy link
Member

@Conengmo Conengmo left a comment

Choose a reason for hiding this comment

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

Seems good to me

@Conengmo
Copy link
Member

Our tests are failing because of some dependency BS. GeoPandas needs Fiona, which needs Boto3 which isn't in its requirements.

geopandas/geopandas#823

Toblerity/Fiona#507

pysal/spaghetti#113

which now surfaces because installing fiona with conda-forge seemed to have started to install the alpha release

@ocefpaf you seem quite involved with conda-forge right, can you help?

@ocefpaf
Copy link
Member

ocefpaf commented Sep 17, 2018

@ocefpaf you seem quite involved with conda-forge

I actually caused that error 😳

can you help?

Done.

@ocefpaf ocefpaf merged commit a044e82 into python-visualization:master Sep 17, 2018
@Conengmo
Copy link
Member

I actually caused that error

Haha :) Well thanks for fixing it!

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.

3 participants