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

fix(material/paginator): prevent keyboard nav to disabled buttons #30627

Merged
merged 1 commit into from
Mar 14, 2025

Conversation

andrewseguin
Copy link
Contributor

Previous and next buttons should not be in the tab order if they are disabled

Fixes b/395610398

Unverified

This user has not yet uploaded their public signing key.
@andrewseguin andrewseguin requested review from crisbeto and mmalerba and removed request for crisbeto March 13, 2025 16:22
@andrewseguin andrewseguin added the dev-app preview When applied, previews of the dev-app are deployed to Firebase label Mar 13, 2025
Copy link

Deployed dev-app for 4301311 to: https://ng-dev-previews-comp--pr-angular-components-30627-dev-ypiu0m2u.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

@andrewseguin andrewseguin added the target: patch This PR is targeted for the next patch release label Mar 14, 2025
@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Mar 14, 2025
@andrewseguin andrewseguin merged commit 75713b0 into angular:main Mar 14, 2025
25 of 27 checks passed
@andrewseguin
Copy link
Contributor Author

The changes were merged into the following branches: main, 19.2.x

andrewseguin added a commit that referenced this pull request Mar 14, 2025

Unverified

This user has not yet uploaded their public signing key.
…0627)

(cherry picked from commit 75713b0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: material/paginator dev-app preview When applied, previews of the dev-app are deployed to Firebase target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants