Skip to content

Post-merge-review: Fix template-require-context-role: align aria-hidden scope and report oc with upstream#2686

Merged
NullVoxPopuli merged 1 commit intoember-cli:masterfrom
johanrd:day_fix/template-require-context-role
Apr 14, 2026
Merged

Post-merge-review: Fix template-require-context-role: align aria-hidden scope and report oc with upstream#2686
NullVoxPopuli merged 1 commit intoember-cli:masterfrom
johanrd:day_fix/template-require-context-role

Conversation

@johanrd
Copy link
Copy Markdown
Contributor

@johanrd johanrd commented Apr 13, 2026

Limit aria-hidden suppression to the immediate parent (port previously walked the full ancestor stack — over-permissive). Report on the role attribute node, not the element, to match upstream.

Cowritten by claude

… loc with upstream

Limit aria-hidden suppression to the immediate parent (port previously
walked the full ancestor stack — over-permissive). Report on the role
attribute node, not the element, to match upstream.
@johanrd johanrd marked this pull request as ready for review April 13, 2026 16:09
@johanrd johanrd force-pushed the day_fix/template-require-context-role branch from 10256eb to 4196ef0 Compare April 13, 2026 17:39
@NullVoxPopuli NullVoxPopuli merged commit 4845395 into ember-cli:master Apr 14, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants