-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Buttons: Don't set a placeholder text color #39034
Conversation
Size Change: -79 B (0%) Total Size: 1.15 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.
I tested this across a variety of themes and it either had no effect (twentynineteen - twentytwentytwo) or fixed things when it was a "dark" theme (Remote, Geologist).
Makes sense to me, but I'm curious as to why this was added in the first place and if we'd be breaking anything by removing it. Did the placeholder text color not always inherit the block's color?
I think so yeah. The fix was added in 2018, so a lot of stuff has happened since then. |
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.
A lot has happened indeed. This is a nice cleanup, thank you.
Description
This reverts https://github.com/WordPress/gutenberg/pull/10160/files, since it's no longer needed. The problem with this fix is that it doesn't account for light buttons in a dark theme. Since now the placeholder inherits the block colors we don't need to set a placeholder color at all, so I think we can just remove these lines.
Testing Instructions
Use remote: Automattic/themes#5574
Add a Button block to a post
Confirm that the placeholder text is still legible.
Screenshots
Before
After
TT2 is unaffected by this change:
Types of changes
Bug fix (non-breaking change which fixes an issue)
Checklist:
*.native.js
files for terms that need renaming or removal).