-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
List View: Update the expanded text color so that it works in both a dark and light context #50434
Conversation
…dark and light context
Size Change: 0 B Total Size: 1.38 MB ℹ️ View Unchanged
|
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!
Flaky tests detected in 6431413. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/4916070317
|
What?
Updates the CSS for list view so that the overrides for button colors works in both a light and dark context.
Why?
This is needed that the list view can work in the browse mode sidebar as well as the regular canvas.
How?
This line of CSS makes these buttons blue by default:
List view already has an override for this to make it grey, but we can change that to just inherit from the wider context, which means that it will use whatever color the rest of the text is.
Testing Instructions
Testing Instructions for Keyboard
As above
Screenshots or screencast