-
-
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
Workaround for MS Edge does not update disabled control's style #28275
Conversation
@MartijnCuppens I haven't tested or reviewed this myself, but likely a candidate for v4 and then bringing back into v5. |
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 really dislike the attribute selector but if it's the only solution we should do this. We'll revert this again once Edge has switched to Chromium.
Does the attribute selector cover the psuedo class for all browsers? |
The
|
Changed the base branch to master. If it's supposed to be backported in 4.x, mark it or something and we'll cherry pick it when we cut the release. |
Fixes #28247
Before: https://codepen.io/anon/pen/VgEmwV
After: https://codepen.io/anon/pen/PVyGLv
This PR targets the components of the Bootstrap. However, if bootstrap team prefer to fix every situation, it can also add the following CSS to the
_reboot.scss
.[disabled] ~ _ {}