test(multiple): check for incorrect usage of Angular Aria directives and log violations#33195
Open
adolgachev wants to merge 8 commits into
Open
test(multiple): check for incorrect usage of Angular Aria directives and log violations#33195adolgachev wants to merge 8 commits into
adolgachev wants to merge 8 commits into
Conversation
angular-robot
Bot
added
detected: feature
adolgachev
force-pushed
the
aria-validation
branch
from
1b0bef9 to
8a745a6
Compare
adolgachev
changed the title
adolgachev
changed the title
adolgachev
force-pushed
the
aria-validation
branch
from
8a745a6 to
be06cbf
Compare
ok7sai
reviewed
May 6, 2026
ok7sai
reviewed
May 6, 2026
adolgachev
force-pushed
the
aria-validation
branch
from
249d896 to
0c9eb7b
Compare
adolgachev
force-pushed
the
aria-validation
branch
from
14258d5 to
e631419
Compare
adolgachev
force-pushed
the
aria-validation
branch
5 times, most recently
from
e17912e to
15a16d9
Compare
adolgachev
added
the
target: rc
adolgachev
commented
May 7, 2026
adolgachev
added
the
action: review
ok7sai
approved these changes
May 8, 2026
| tab: this._tabPattern, | ||
| }); | ||
|
|
||
| private readonly _tabContent = contentChild(TabContent); |
Member
There was a problem hiding this comment.
Should probably consolidate this into the DeferredContent and check whether a DeferredContent is missing when DeferredContentAware is used.
Contributor
Author
There was a problem hiding this comment.
I was thinking since DeferredContent is private, better to show violation tied to TabPanel. Also putting it lower would require adding more the DeferredContentAware which seemed less ideal just for the violation check.
…es and log violations
adolgachev
force-pushed
the
aria-validation
branch
4 times, most recently
from
76dfcb5 to
5b722ad
Compare
…nd log violations
…nd log violations
adolgachev
force-pushed
the
aria-validation
branch
from
5b722ad to
45490f3
Compare
…g element and violations
adolgachev
force-pushed
the
aria-validation
branch
from
45490f3 to
e40eec9
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Used gemini to go through all of the components and generate additional violation checks for possible issues with how each set of angular aria directives are used together. Also generated specs for the new and previously existing violations.
Final commit adds a utility method to report the violations which does the console.error but also takes in an element to help in location where the issue is coming from.