Make WordPress Core

Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#57222 closed defect (bug) (fixed)

Improve Supported Versions table accessibility in `SECURITY.md`

Reported by: audrasjb's profile audrasjb Owned by: peterwilsoncc's profile peterwilsoncc
Milestone: 6.2 Priority: normal
Severity: normal Version:
Component: Security Keywords: has-patch
Focuses: accessibility, docs Cc:

Description (last modified by audrasjb)

In SECURITY.md, the list of supported versions uses emojis to indicate whether a branch is supported or not. For better accessibility, it would be nice to add some text to complete the image representation with a text.

I propose to just add "Yes" or "No" after the emoji so we have both a visual representation and a simple, clear text.

Change History (10)

#1 @audrasjb
2 years ago

Note: This is a follow-up to a quick conversation with @peterwilsoncc on this PR: https://github.com/WordPress/wordpress-develop/pull/3692#pullrequestreview-1195408087

#2 @audrasjb
2 years ago

  • Description modified (diff)

This ticket was mentioned in PR #3694 on WordPress/wordpress-develop by @audrasjb.


2 years ago
#3

  • Keywords has-patch added

@peterwilsoncc commented on PR #3694:


2 years ago
#4

This looks good to me.

In your experience do you know if emoji are problematic for screen readers? The question behind the question is will screen reader users hear the first row as "six point one point ex white check mark yes"?

If a simple "yes" will be better for accessibility, I am happy to just go with that.

@audrasjb commented on PR #3694:


2 years ago
#5

Good question @peterwilsoncc :)
It seems that assistive technologies will restitute ✅ as "White Heavy Check Mark".
The recommendation is to avoid using emojis as a replacement for text.

But it's also recommended to place the textual information before the emoji…
Shouldn't we completely remove the emojis from this documentation?

@peterwilsoncc commented on PR #3694:


2 years ago
#6

Shouldn't we completely remove the emojis from this documentation?

Yeah, let's do that. It's unambiguously accessible.

@audrasjb commented on PR #3694:


2 years ago
#7

@peterwilsoncc I replaced all emojis with plain text words. Really, I think it doesn't hurt :)

#8 @peterwilsoncc
2 years ago

  • Milestone changed from Awaiting Review to 6.2

#9 @peterwilsoncc
2 years ago

  • Owner set to peterwilsoncc
  • Resolution set to fixed
  • Status changed from new to closed

In 54919:

Security: Improve accessibility of security policy.

Replace emoji indicators of support with "yes" and "no". This prevents "White Heavy Check Mark" indicating supported versions for screen-reader users.

Props audrasjb.
Fixes #57222.

Note: See TracTickets for help on using tickets.