Skip to content

Commit 7122c53

Browse files
committed
docs: address Copilot review wording (PR #19)
1 parent 2bebc6e commit 7122c53

1 file changed

Lines changed: 7 additions & 5 deletions

File tree

docs/rules/template-no-aria-hidden-on-focusable.md

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -49,11 +49,13 @@ This rule **allows** the following:
4949

5050
## Caveats
5151

52-
Component invocations, argument/`this`/path-based tags, and namespace-pathed
53-
tags are "opaque" — we can't statically know what they render. The descendant
54-
check skips these branches to avoid false positives. If a component renders a
55-
focusable element beneath an `aria-hidden` ancestor, the keyboard trap still
56-
exists at runtime; this rule can't detect it.
52+
Component invocations, argument/`this`/path-based tags, namespace-pathed tags,
53+
and hyphenated custom elements (e.g. `<my-element>`) are "opaque" — we can't
54+
statically know what they render. The descendant check skips these branches
55+
(via `isNativeElement`, which gates on the HTML/SVG/MathML tag lists and on
56+
scope references) to avoid false positives. If a component renders a focusable
57+
element beneath an `aria-hidden` ancestor, the keyboard trap still exists at
58+
runtime; this rule can't detect it.
5759

5860
Dynamic content inside `{{...}}` mustache statements is similarly not inspected.
5961

0 commit comments

Comments
 (0)