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

More sliders #932

Merged
merged 20 commits into from
Jul 2, 2018
Merged

More sliders #932

merged 20 commits into from
Jul 2, 2018

Conversation

cchaos
Copy link
Contributor

@cchaos cchaos commented Jun 19, 2018

This PR fleshes out the EuiRange control to align with some requests for more options around styling.

Some permutations created in the docs:
screen shot 2018-06-19 at 20 11 57 pm


  • Browser tested in Chrome, FF, IE11
  • Accessibility
  • Mobile

I would like a pair on this if anyone is available. It's a combo of JS and CSS/design heavy stuff.

This will be able to go in on its own, but in order to address some more custom labeled versions, I'll pass that on to @chandlerprall. But all the styles needed for that have been created in this.

@cchaos
Copy link
Contributor Author

cchaos commented Jun 21, 2018

@chandlerprall I'm not sure if that last commit made it any more readable. If you have ideas, please feel free to make edits directly.

@cchaos
Copy link
Contributor Author

cchaos commented Jul 2, 2018

@chandlerprall Can I please get final looks on this?

Copy link
Contributor

@chandlerprall chandlerprall left a comment

Choose a reason for hiding this comment

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

LGTM! Easy to read/follow. Pulled and played with the docs examples and they're fantastic.

@cchaos cchaos merged commit df47f15 into elastic:master Jul 2, 2018
@cchaos cchaos deleted the more-sliders branch July 2, 2018 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants