-
Notifications
You must be signed in to change notification settings - Fork 78
Add 2.1.3 to iframe focusable rule #2229
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't that a case where Scenario 1 (the rule is not as strict as a requirement) of Secondary requirements applies?
That is, anything that fails 2.1.1 also automatically fails 2.1.3, similarly to the 1.4.3/1.4.6 situation.
Additionally, 2.1.3 is a AAA criterion, so having this as a primary requirements means that tools that only check for level AA will never be able to have a consistent implementation of the rule (which was pretty much what led us to introduce secondary requirements for the 1.4.3/1.4.6 or 2.4.6/2.4.9 cases: initial PR, linking to more issues in the wcag-act-rules repo)
|
@WilcoFiers another issue related to this rule. |
I forgot to put the call for review label on this, but that e-mail went out on April 10th. That's over two weeks ago, so I'm merging this PR. |
Closes #2026
IMPORTANT: This is going to cause implementors to be inconsistent. That's why I'm using a 2 week review. It is incorrect for this rule not to fail 2.1.3 for the same reason that Scrollable content can be reached with sequential focus navigation fails 2.1.3. The only difference between 2.1.1 and 2.1.3 is an exception for path based input. That's not the case for this iframes or scrollable regions.
Need for Call for Review: 2 weeks
How to Review And Approve