Skip to content

Conversation

crisbeto
Copy link
Member

@crisbeto crisbeto commented Feb 3, 2022

When preserveContent is enabled we add an extra display: none to prevent users from tabbing into hidden content. The selector was placed on the wrong element for the MDC-based tabs.

…n preserveContent is enabled

When `preserveContent` is enabled we add an extra `display: none` to prevent users from tabbing into hidden content. The selector was placed on the wrong element for the MDC-based tabs.
@crisbeto crisbeto added P2 The issue is important to a large percentage of users, with a workaround target: patch This PR is targeted for the next patch release labels Feb 3, 2022
@crisbeto crisbeto requested a review from devversion February 3, 2022 09:43
@crisbeto crisbeto added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release and removed target: patch This PR is targeted for the next patch release labels Feb 3, 2022
@crisbeto crisbeto merged commit 02d6e7d into angular:master Feb 3, 2022
amysorto pushed a commit to amysorto/components that referenced this pull request Feb 15, 2022
…n preserveContent is enabled (angular#24346)

When `preserveContent` is enabled we add an extra `display: none` to prevent users from tabbing into hidden content. The selector was placed on the wrong element for the MDC-based tabs.
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Mar 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker P2 The issue is important to a large percentage of users, with a workaround target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants