-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Add "black" and "white" color options to default color palette. #22082
Conversation
Removes other intermediary grays from the UI but keeps them in the CSS declarations as classes for back-compat.
Size Change: +54 B (0%) Total Size: 819 kB
ℹ️ View Unchanged
|
What happens when one of the previous colors we are deprecating is used is that the block keeps the color in the editor and frontend but none of the current colors we have appear as selected (as the select color is now not available). |
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 👍
Let's add black and white as default color options since they have ubiquitous use and value for more accessible color contrast with the widest variety of colors:
To keep the palette balanced this change also drops some of the intermediary grays from the UI — they are kept, though, in the CSS declaration to ensure the CSS can still be applied. Need to revise if a former class being set causes any issues in the UI when loaded again cc @jorgefilipecosta