-
-
Notifications
You must be signed in to change notification settings - Fork 78.9k
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
Add note that user-select-all
is not supported on IE or Edge Legacy
#32120
Conversation
… or Edge Legacy Fixes: #32092 I kept the style/formatting of the compatibility issue **bold**, so it matches what is used elsewhere for IE issues, e.g.: https://getbootstrap.com/docs/4.5/utilities/flex/#auto-margins https://getbootstrap.com/docs/4.5/utilities/position/#sticky-top
Thanks! @mdo LGTY or do you think we should use a (warning) callout? I think we are better off using a warning callout |
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'm ok with the bold warning
@coliff we can't merge this it seems; you removed your fork at some point :/ |
@@ -16,4 +16,6 @@ Change how the content is selected when the user interacts with it. | |||
{% endcapture %} | |||
{% include example.html content=example %} | |||
|
|||
**Note: `user-select-all` is not supported on Internet Explorer or Edge Legacy.** |
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'd move this up to be part of Line 10, and make this edit:
Note that Internet Explorer and Legacy Edge have no support for the
all
value foruser-select
, and as such,.user-select-all
is not supported by either browser.
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.
@mdo can you make a PR please? ❤
Fixes: #32092
I kept the style/formatting of the compatibility issue bold, so it matches what is used elsewhere for IE issues, e.g.:
https://getbootstrap.com/docs/4.5/utilities/flex/#auto-margins
https://getbootstrap.com/docs/4.5/utilities/position/#sticky-top
I haven't been able to test this change locally by building the docs (don't have Jekyll 4.x installed right now)... but as this is simple change hopefully it should be ok.
Preview: https://deploy-preview-32120--twbs-bootstrap.netlify.app/docs/4.5/utilities/interactions/