Skip to content

fix(multiple): prevent focus on disabled components #32263

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 5, 2025
Merged

fix(multiple): prevent focus on disabled components #32263

merged 1 commit into from
Nov 5, 2025

Conversation

@tjshiu
Copy link
Contributor

@tjshiu tjshiu commented Nov 5, 2025

  • Whan a component is completely disabled - only the parent component is focusable even when softdisabled=true.

@tjshiu tjshiu requested a review from a team as a code owner November 5, 2025 21:53
@tjshiu tjshiu requested review from crisbeto, ok7sai and wagnermaciel and removed request for a team, crisbeto and wagnermaciel November 5, 2025 21:53
@tjshiu tjshiu added Accessibility This issue is related to accessibility (a11y) area: cdk/a11y target: rc This PR is targeted for the next release-candidate action: merge The PR is ready for merge by the caretaker labels Nov 5, 2025
@tjshiu tjshiu removed the request for review from wagnermaciel November 5, 2025 22:31
@tjshiu tjshiu merged commit 4aad878 into angular:main Nov 5, 2025
30 of 32 checks passed
@tjshiu
Copy link
Contributor Author

tjshiu commented Nov 5, 2025

This PR was merged into the repository. The changes were merged into the following branches:

tjshiu added a commit that referenced this pull request Nov 5, 2025
@tjshiu
fix(multiple): prevent focus on disabled components (#32263)
caa2b3b 
(cherry picked from commit 4aad878)
adolgachev pushed a commit to adolgachev/components that referenced this pull request Nov 6, 2025
@tjshiu @adolgachev
fix(multiple): prevent focus on disabled components (angular#32263)
2acf62b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ok7sai ok7sai ok7sai approved these changes

Assignees

No one assigned

Labels

Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker area: cdk/a11y target: rc This PR is targeted for the next release-candidate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tjshiu @ok7sai