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
[palette] Standardize the secondary color #9918
[palette] Standardize the secondary color #9918
Conversation
This exactly fixes the issue I wanted to submit :) |
Humm, looking at the other component, I have the feeling that we should be updating the other components too to be consitant. |
Just had a look - the spec is inconsistent. This is light, for example: https://material.io/guidelines/components/buttons.html#buttons-dropdown-buttons But in examples 3, 5 & 6 of the color picker, the FAB is dark: https://material.io/color/#!/?primary.color=2196F3&secondary.color=E91E63&view.left=0&view.right=0 |
1bab5f2
to
09a4aa2
Compare
09a4aa2
to
011ebbf
Compare
I have completed the full-scale change. |
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
Looking at the different resources I can gather:
There is no reason for the secondary color to use different color hues than the primary color.
Fix @mbrookes's concern in #9913 (comment)
Breaking change
The secondary color now behaves the same way than the other colors (primary, error). We always use the
main
tone by default instead of thelight
tone.It's unclear if this change is making the implementation follow the specification more closely. The direct win is simplicity and predictability.