Skip to content

[colors] Fix A color inconsistencies#26196

Merged
oliviertassinari merged 1 commit intomui:nextfrom
oliviertassinari:fix-color-order
May 10, 2021
Merged

[colors] Fix A color inconsistencies#26196
oliviertassinari merged 1 commit intomui:nextfrom
oliviertassinari:fix-color-order

Conversation

@oliviertassinari
Copy link
Copy Markdown
Member

@oliviertassinari oliviertassinari commented May 8, 2021

A follow-up based on @Angelk90's feedback in #26015 (comment).

When Material Design doesn't provide the colors of A100, A200, A400, A700, we make them correspond with values of 100, 200, 400, 700.

This is done correctly in brown and blueGrey, however, we didn't do it correctly with grey. I'm fixing it.

@oliviertassinari oliviertassinari added the design: material This is about Material Design, please involve a visual or UX designer in the process label May 8, 2021
@mui-pr-bot
Copy link
Copy Markdown

mui-pr-bot commented May 8, 2021

Details of bundle changes (experimental)

Generated by 🚫 dangerJS against d6619e2

@eps1lon
Copy link
Copy Markdown
Member

eps1lon commented May 10, 2021

A description what is happening would be nice.

@oliviertassinari
Copy link
Copy Markdown
Member Author

Updated

@mnajdova mnajdova changed the title [colors] Fix A color inconsistencies [colors] Fix a color inconsistencies May 10, 2021
@mnajdova mnajdova changed the title [colors] Fix a color inconsistencies [colors] Fix A color inconsistencies May 10, 2021
@oliviertassinari oliviertassinari merged commit 0188c64 into mui:next May 10, 2021
@oliviertassinari oliviertassinari deleted the fix-color-order branch May 10, 2021 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

design: material This is about Material Design, please involve a visual or UX designer in the process

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants