Update aria-busy to reflect relaxed accessibility child constraints when true - #2754
Update aria-busy to reflect relaxed accessibility child constraints when true#2754giacomo-petri wants to merge 4 commits into
Conversation
✅ Deploy Preview for wai-aria ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
| branches are modified. | ||
| If changes to a rendered <rref>widget</rref> would create a state where the <rref>widget</rref> is modifying <a>accessibility children</a> during script | ||
| execution, authors MAY set <code>aria-busy</code> to <code>true</code> on the <rref>widget</rref> during the update process. While <code>aria-busy</code> is set to <code>true</code>, | ||
| the <rref>widget</rref> MAY temporarily contain invalid <a>accessibility children</a>. Authors MUST set <code>aria-busy</code> back to <code>false</code> once the update is complete. |
There was a problem hiding this comment.
The new MUST statement seems problematic to me. It does not seem testable since "once the update is complete" is an author decision.
I think it's more important to explain what happens when authors do not do this (e.g. AT may ignore the content marked busy).
I think a note, possibly including a warning on potential WCAG failures.
There was a problem hiding this comment.
In that case, I'd move the example into the note.
There was a problem hiding this comment.
I agree that the author MUST is not testable with an automated static or generic test. However, I believe it is testable. I proposed some revisions to the wording that I believe could improve clarity and thus testability.
I think a clear, normative authoring requirement is of this nature is important.
| branches are modified. | ||
| If changes to a rendered <rref>widget</rref> would create a state where the <rref>widget</rref> is modifying <a>accessibility children</a> during script | ||
| execution, authors MAY set <code>aria-busy</code> to <code>true</code> on the <rref>widget</rref> during the update process. While <code>aria-busy</code> is set to <code>true</code>, | ||
| the <rref>widget</rref> MAY temporarily contain invalid <a>accessibility children</a>. Authors MUST set <code>aria-busy</code> back to <code>false</code> once the update is complete. |
There was a problem hiding this comment.
I agree that the author MUST is not testable with an automated static or generic test. However, I believe it is testable. I proposed some revisions to the wording that I believe could improve clarity and thus testability.
I think a clear, normative authoring requirement is of this nature is important.
Co-authored-by: Matt King <[email protected]>
|
@mcking65, I've incorporated your suggestions. I think it should be good now. @pkra, the MUST is still there, but I agree with @mcking65 on this point. |
Taking myself off this PR. I won't have time for this in the near future.
|
@mcking65 can you please re-review |
5 participants
🚀 Netlify Preview:
🔄 this PR updates the following sspecs:
Closes #2737
This PR updates
aria-busycontent:aria-busyis set totrue, permitting a temporarily non-conformant accessibility child structure during the update process, as discussed in the ARIA WG meetingTest, Documentation and Implementation tracking
Once this PR has been reviewed and has consensus from the working group, tests should be written and issues should be opened on browsers. Add N/A and check when not applicable.