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

789 slider styling improvements #1005

Merged
merged 13 commits into from
Jul 7, 2016
Merged

Conversation

aeschylus
Copy link
Collaborator

Styles sliders on image manipulation controls so they appear underneath the caret of their respective icons. Displays the percentage represented by the slider to the right of the handle. Updates on reset.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.9%) to 44.092% when pulling 451615a on 789-slider-styling-improvements into 2688f8c on release2.1.

@aeschylus
Copy link
Collaborator Author

Hm. I need to write some tests for this.

@coveralls
Copy link

coveralls commented Jul 7, 2016

Coverage Status

Coverage decreased (-1.0%) to 44.082% when pulling ef61a86 on 789-slider-styling-improvements into 2688f8c on release2.1.

@rsinghal
Copy link
Collaborator

rsinghal commented Jul 7, 2016

Yes, we need to write tests, but we changed different widgets, which will get significantly modified by iiifManifestLayouts, which is why we haven't added tests yet.

@rsinghal rsinghal merged commit 8bc3982 into release2.1 Jul 7, 2016
@rsinghal rsinghal deleted the 789-slider-styling-improvements branch July 7, 2016 23:37
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.

4 participants