Make WordPress Core

Opened 20 months ago

Closed 20 months ago

Last modified 20 months 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
20 months 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
20 months ago

  • Description modified (diff)

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


20 months ago
#3

  • Keywords has-patch added

@peterwilsoncc commented on PR #3694:


20 months 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:


20 months 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:


20 months 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:


20 months ago
#7

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

#8 @peterwilsoncc
20 months ago

  • Milestone changed from Awaiting Review to 6.2

#9 @peterwilsoncc
20 months 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.