Skip to content
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

Closed
wants to merge 1 commit into from
Closed

Add note that user-select-all is not supported on IE or Edge Legacy #32120

wants to merge 1 commit into from

Conversation

coliff
Copy link
Contributor

@coliff coliff commented Nov 10, 2020

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/

… 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
@XhmikosR
Copy link
Member

XhmikosR commented Nov 10, 2020

Thanks!

@mdo LGTY or do you think we should use a (warning) callout? I think we are better off using a warning callout

@XhmikosR XhmikosR added this to Inbox in v4.6.0 via automation Nov 10, 2020
@XhmikosR XhmikosR requested a review from mdo November 12, 2020 07:51
@XhmikosR XhmikosR linked an issue Nov 12, 2020 that may be closed by this pull request
Copy link
Member

@MartijnCuppens MartijnCuppens left a 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

@XhmikosR
Copy link
Member

@coliff we can't merge this it seems; you removed your fork at some point :/

@XhmikosR XhmikosR closed this Nov 13, 2020
@XhmikosR XhmikosR removed this from Inbox in v4.6.0 Nov 13, 2020
@@ -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.**
Copy link
Member

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 for user-select, and as such, .user-select-all is not supported by either browser.

Copy link
Member

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? ❤

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User-Select: all is not supported in IE or Edge Legacy
4 participants