-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
Update placeholder text color for standard themes #53719
base: trunk
Are you sure you want to change the base?
Conversation
Size Change: 0 B Total Size: 1.51 MB ℹ️ View Unchanged
|
Hey @derekblank 👋 thanks for working on this task!
It looks like this theme is a block-based theme and not a standard theme (it has a theme.json file). You could use the Seedlet theme, for example, which is a standard theme. What we'd need to change is the default SCSS styles for the placeholders in the RichText component ( Thanks! |
Ah, thanks for the info! I'll check there using this theme. |
…heme-placeholder-text
@geriux I tested these changes with the Seedlet theme, and also the Russell theme, which was the theme that Chris initially reported the darker placeholder color. With the changes from this branch applied, the placeholder text color appears to display correctly: Placeholder.movI described the strategy for this change a bit further in this comment, as the gray colors used in |
What?
Fixes placeholder text colors for standard themes.
Why?
Currently for sites using a standard theme like Russell, there is no distinction between a text input's primary color and its placeholder color.
How?
Removes the usage of the
globalStyles.text
color as also functioning as the placeholder text color.Testing Instructions
Screenshots or screencast