-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Modal headings and labels missing editor text color #71311
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
base: trunk
Are you sure you want to change the base?
Modal headings and labels missing editor text color #71311
Conversation
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.
To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
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.
Thank you for working on this ✨
Looks good overall! I noticed that the h2 and h3 elements inside the modal aren’t picking up the correct color. I’m wondering if it might make sense to target the .components-modal__frame
class to set the text color there, and also explicitly style h1, h2, and h3 within it to override the defaults coming from the common CSS.
modal-colors.mp4
@juanfra I have updated the changes to target the |
@kushagra-goyal-14, could you rebase this branch on top of the latest trunk. That should resolve CI checks. |
Thanks for working on this @kushagra-goyal-14! The code changes look good. Not sure if the comment before the color is a bit redundant. Can you please be sure you sync trunk and rebase with the latest changes to fix the CI checks? |
What?
Closes #50448
This PR updates the Modal Heading and Label text color value so that it matches with the text color used for other areas of the editor. Previously it was using color from wp-admin common.css.
Why?
How?
This PR is refreshes and is inspired from - #51246 and #50916
Testing Instructions
Open the WordPress block editor
Open the Preferences modal and check the colour value using Inspect Element.
Testing Instructions for Keyboard
Screenshots or screencast