Skip to content
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

Tech using role log #435

Merged
merged 10 commits into from
Oct 2, 2018
Merged

Tech using role log #435

merged 10 commits into from
Oct 2, 2018

Conversation

mbgower
Copy link
Contributor

@mbgower mbgower commented Jul 18, 2018

new technique for 4.1.3 Status Messages
https://rawgit.com/w3c/wcag/tech-using-role-log/techniques/aria/aria-log-role.html

Links go to working examples which have a separate pull request #436

@mbgower mbgower mentioned this pull request Jul 18, 2018
@mbgower mbgower added Techniques Ready for initial review A new technique ready for +1s or itterations labels Jul 18, 2018
@rachaelbradley rachaelbradley self-requested a review July 24, 2018 15:23
@DavidMacDonald
Copy link
Contributor

  • It appears the first example once the "send" button is sent gives "undefined" if it is hit more than once.
  • The default value for aria-relevant is "additions text" so it doesn't need to be added to the second example. Perhaps a description of the default behaviour would work there.

When the aria-relevant attribute is not provided, the default value, additions text, indicates that text modifications and node additions are relevant, but that node removals are irrelevant.

https://www.w3.org/TR/wai-aria-1.1/#aria-relevant

@mraccess77
Copy link

Some have even suggested that we should avoid using aria-relevant altogether.

https://medium.com/dev-channel/why-authors-should-avoid-aria-relevant-5d3164fab1e3

@mbgower
Copy link
Contributor Author

mbgower commented Jul 31, 2018

re: aria-relevant

I can take this out for now. I do think there is a case to be made for using it when a 'busy' message disappears (i.e., a user with vision sees the busy message go away and starts working, but the blind user is never notified of the removal of 'busy' so just waits for an update that never is announced), so I may tackle it when I get to that technique/use case -- although I agree the AT support is spotty right now.

@mbgower
Copy link
Contributor Author

mbgower commented Aug 2, 2018

It appears the first example once the "send" button is sent gives "undefined" if it is hit more than once.

Disabled Send button to make it one-time in #436

@alastc
Copy link
Contributor

alastc commented Aug 14, 2018

Using role=log to improve AT support

Should the main heading talk about "improve AT support"?

How about: "Using role=log to identify sequential information updates"

Also, I might be misunderstanding aria-atomic but wouldn't 'false' mean that it reads the whole thing at once? That seems to be the opposite of what is desirable here.

Also, the procedure seems self-fulfilling, where a log is marked up as a log. Could it be something like:

On a page that contains sequentially updating information:

  1. Check that the container of the information is given a role of log.

modified title and test procedure.
@mbgower
Copy link
Contributor Author

mbgower commented Sep 24, 2018

@alastc in regard to your question about aria-atomic, if it is set to false (the default for log) then it will only read the changes, not the entire contents. So your understanding seems to be opposite to the implementation. The following is from aria 1.1 spec:

If none of the ancestors have explicitly set aria-atomic, the default is that aria-atomic is false, and assistive technologies will only present the changed node to the user.
If aria-atomic is explicitly set to false, assistive technologies will stop searching up the ancestor chain and present only the changed node to the user.
If aria-atomic is explicitly set to true, assistive technologies will present the entire contents of the element, including the author-defined live region label if one exists.

Made your suggested changes to the title and test procedure wording.

@alastc
Copy link
Contributor

alastc commented Sep 25, 2018

Thanks Mike, I haven't used atomic in anger yet.

I just assumed that 'atomic' meant peice by peice, therefore 'true' would mean per-change, and false would mean the whole lot. I shall update my mental model ;-)

@alastc alastc added Ready for WG Review and removed Ready for initial review A new technique ready for +1s or itterations labels Sep 26, 2018
@awkawk awkawk merged commit cb2f8ba into master Oct 2, 2018
@awkawk awkawk deleted the tech-using-role-log branch October 2, 2018 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants