Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#57222 closed defect (bug) (fixed)

Improve Supported Versions table accessibility in `SECURITY.md`

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

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
4 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
4 years ago

  • Description modified (diff)

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


4 years ago
#3

  • Keywords has-patch added

​@peterwilsoncc commented on ​PR #3694:


4 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:


4 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:


4 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:


4 years ago
#7

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

#8 @peterwilsoncc
4 years ago

  • Milestone Awaiting Review → 6.2

#9 @peterwilsoncc
4 years ago

  • Owner set to peterwilsoncc
  • Resolution → fixed
  • Status new → 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.