-
Notifications
You must be signed in to change notification settings - Fork 142
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
[Demo] Update features example dashboard with selectors #333
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome as usual! 🧹 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. 🚀
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! I think we should improve the Slider
example if easy to do so, but no worries if not.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm! If I could choose, I would probably work more with a graph, and ideally actually only one component, but that may be tough with fitting all the options in.
Yeah, that's what I had before, but it did not work well if we want to have all selectors on one page 😄 They are contradicting each other / intersecting - which made the interactions horrible with one graph. |
Description
Screenshot
Notice
I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":