Skip to content

[aria/multiple] Add to and clean up input validation across aria components#32713

Merged
adolgachev merged 2 commits intoangular:mainfrom
adolgachev:aria-validation
Jan 28, 2026
Merged

[aria/multiple] Add to and clean up input validation across aria components#32713
adolgachev merged 2 commits intoangular:mainfrom
adolgachev:aria-validation

Conversation

@adolgachev
Copy link
Copy Markdown
Contributor

@adolgachev adolgachev commented Jan 27, 2026

Added one more validation check for tree when setting multiple values on a single select.
Removed validate method with no checks.

Went through other components and didn't see a practical need for other checks on inputs.

Will do another run-through to see if other mistakes can be validated.

@adolgachev adolgachev changed the title [aria/multiple] Fix up validation [aria/multiple] Add to and clean up input validation across aria components Jan 27, 2026
@adolgachev adolgachev marked this pull request as ready for review January 27, 2026 21:46
@adolgachev adolgachev requested review from tjshiu and removed request for andrewseguin and wagnermaciel January 27, 2026 21:47
@pullapprove pullapprove Bot requested a review from wagnermaciel January 27, 2026 21:47
@adolgachev adolgachev requested review from ok7sai and removed request for wagnermaciel January 27, 2026 21:47
@adolgachev adolgachev added Accessibility This issue is related to accessibility (a11y) action: review The PR is still awaiting reviews from at least one requested reviewer action: presubmit The PR is in need of a google3 presubmit target: patch This PR is targeted for the next patch release labels Jan 27, 2026
@adolgachev adolgachev added action: merge The PR is ready for merge by the caretaker and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jan 28, 2026
@adolgachev adolgachev removed the request for review from ok7sai January 28, 2026 21:39
@adolgachev adolgachev removed the action: presubmit The PR is in need of a google3 presubmit label Jan 28, 2026
@adolgachev adolgachev merged commit dbb94dc into angular:main Jan 28, 2026
38 of 40 checks passed
@adolgachev
Copy link
Copy Markdown
Contributor Author

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

adolgachev added a commit that referenced this pull request Jan 28, 2026
…onents (#32713)

* fix(aria/toolbar): remove unnnecessary validation methods

* fix(aria/tree): add validation for single selection tree with multiple values

(cherry picked from commit dbb94dc)
@adolgachev adolgachev deleted the aria-validation branch January 28, 2026 21:50
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Feb 28, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Accessibility This issue is related to accessibility (a11y) action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants