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 modifier props! #2333
More modifier props! #2333
Conversation
NumericInput fill propPreview: documentation | landing | table |
{ | ||
[Classes.ACTIVE]: active, | ||
[Classes.INTENT_PRIMARY]: active && intentClass == null, |
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.
isn't this too opinionated? seems like a visual change that could be separate from this PR?
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.
@llorca this is your opinion! when we changed the hover state to gray, we had to go add this intent-primary for the active "selected" state. so this visual change was made long ago.
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.
code looks good once the build is green
use table-dev-app nav propsPreview: documentation | landing | table |
Follow up from #2329
Tag
: minimalMenuItem
: activeTagInput
: fill, largeNumericInput
: fill