-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
Change floating label placeholder's color #39494
Change floating label placeholder's color #39494
Conversation
👍 Yes! I've often been confused by the current text color since it looks like a value rather than a placeholder. |
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.
The solution looks fine from a CSS point of view. Any thoughts @mdo ?
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.
Do we need to modify the $form-floating-transition
value as well?
I think it's fine as-is. |
Is there anything preventing this from being merged? |
For the moment, the target of this one is v5.4.0. The next version will probably be a v5.3.x. We'll evaluate when to merge it at the appropriate time. |
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 think we can add this in v5.3.4.
Description
Changed the floating labels placeholder's text color from black to light gray (
opacity: 0.65
).I removed the styling from the other
label
tags under.form-select
and.form-control:-webkit-autofill
parents as everylabel
should now have the same behavior.Motivation & Context
As requested in issue #39420
Type of changes
Checklist
npm run lint
)Live previews
https://deploy-preview-39494--twbs-bootstrap.netlify.app/docs/5.3/forms/floating-labels/#example/
Related issues
#39420