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

Keep layer time slider under legend #2205

Merged
merged 1 commit into from
Aug 30, 2017
Merged

Conversation

mmcfarland
Copy link
Contributor

Overview

The z-index of the slider kept the slider above the legend, obscuring
it. This new value keeps it above the map controls but below the
legend.

Connects #2018

Demo

Before

screenshot from 2017-08-29 14 37 39

After

screenshot from 2017-08-29 14 46 57

Notes

A quick fix for the recently merged UI component.

Testing Instructions

  • Ensure the slider is appropriately displayed when compared to neighboring UI elements like a legend.

The z-index of the slider kept the slider above the legend, obscuring
it.  This new value keeps it above the map controls but below the
legend.
Copy link

@arottersman arottersman left a comment

Choose a reason for hiding this comment

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

+1, tested. Sorry for not catching that on the original.

@mmcfarland mmcfarland merged commit 8686ef2 into develop Aug 30, 2017
@mmcfarland mmcfarland deleted the mjm/climate-slide-legend-z branch August 30, 2017 19:18
@rajadain rajadain mentioned this pull request Oct 16, 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.

None yet

3 participants