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

Spike: Screen reader audit of phone numbers #13951

Closed
3 of 6 tasks
Tracked by #14080
laflannery opened this issue May 31, 2023 · 3 comments
Closed
3 of 6 tasks
Tracked by #14080

Spike: Screen reader audit of phone numbers #13951

laflannery opened this issue May 31, 2023 · 3 comments
Assignees
Labels
accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops

Comments

@laflannery
Copy link
Contributor

laflannery commented May 31, 2023

Description

Audit of screen reader behavior regarding phone numbers to determine which screen readers (if any) read phone numbers correctly even if there is not an aria-label added.

Screen readers to test

  • Jaws
  • NVDA
  • Windows Narrator
  • VoiceOver

Acceptance Criteria

  • Common screen readers are tested
  • Results documented

Team

Please check the team(s) that will do this work.

  • CMS Team
  • Public Websites
  • Facilities
  • User support
@laflannery laflannery added the Needs refining Issue status label May 31, 2023
@github-actions github-actions bot added the CMS Team CMS Product team that manages both editor exp and devops label May 31, 2023
@laflannery laflannery self-assigned this May 31, 2023
@laflannery laflannery added the accessibility Issues related to accessibility label May 31, 2023
@productmike productmike removed the Needs refining Issue status label Jun 5, 2023
@productmike productmike changed the title Screen reader audit of phone numbers Spike: Screen reader audit of phone numbers Jun 5, 2023
@sara-amanda
Copy link

CAIA-Related Inquiry

  • Contact: Randi Hecht
  • Topic: Long-Term Solutions, Short-Term Solutions, and Future Considerations, Relating to: ARIA labels applied to telephone numbers in Drupal

Long-Term Recommendation

The long-term recommendation would be to avoid using ARIA, per CAIA A11ys.

Rationale

  • If the link already has a number the aria label isn't needed. The aria label would be duplicating the actual link. This action (even though small.) Could cause confusion to some screen reader users.
  • Where the label would work is if there were multiple numbers for different countries. Then the case could be made for that label stating names of the various countries.

Issue Complicating Things

  • Issue: We add aria labels to all phone numbers in Drupal because they aren't automatically added.
  • Historical: The reason this was done in the first place was because if you put a phone number in the SR would read "nine hundred and eighteen" instead of "nine one eight"

Short-Term Recommendation

While Laura F. finishes testing, she is recommending that we continue the use of the aria labels as the style guide instruct, until they are officially changed.

  • Testing should be completed in a few sprints from now.
  • She is seeing evidence that they are no longer being needed, but would like to validate further.

Example Formatting for Short-Term Updates

Original:
<a href="tel:+19187815678" aria-label="1.9 1 8. 7 8 1. 5 6 7 8.">918-781-5678</a>

One change to this in the aria-label and have this be:
+ 1. 9 1 8. 7 8 1. 5 6 7 8.

  • Do this instead because without that it's a different phone number so we want to make sure screen reader folks get that information
  • Then in order to actually add this all into Drupal, you will need to use the source view in the rich text editor where you are adding this.

Future Considerations

  • Audit Needed: If we wind up not doing these anymore, we would need to figure out how to remove them from all the numbers that are currently set up with aria labels in Drupal.
  • Improvement: If we no longer will have to add these in manually at a later date that will eliminate an added step.

@laflannery
Copy link
Contributor Author

@sara-amanda Couple other things to keep everything in one place and for awareness:

  • I have another ticket for a CMS phone number audit and the hope was to see how many actually are using aria (and following the other guidelines) But this would hopefully tell you how many did need to get updated
  • If we did end up with a recommendation to no longer use the aria-label:
    • We would want to work with the DST to update the component to remove aria from there was well so everything was consistent
    • This would help to get rid of the false positive that keeps coming up in Siteimprove for our editors so I would love it
    • There might also be some other hardcoded phone numbers that maybe are using aria that are older so aren't using the component that we would want to update

@laflannery
Copy link
Contributor Author

laflannery commented Jun 21, 2023

Testing results spreadsheet

Not present in the results because I don't have the devices:

  • Mac
  • Android

Summary

  • In general, phone numbers are read smoother when an aria label is included but this doesn't necessarily make them wrong
  • TTY number on Desktop was read "wrong", as seven hundred and eleven instead of individual numbers
  • Extensions were also read "wrong" in multiple instances, as the full number instead of individual numbers
  • 800 was sometimes read as "eight hundred" but that is a known convention

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accessibility Issues related to accessibility CMS Team CMS Product team that manages both editor exp and devops
Projects
None yet
Development

No branches or pull requests

3 participants