ckeditor5 icon indicating copy to clipboard operation
ckeditor5 copied to clipboard

Type Around feature is missing aria tags to be read properly by screen-readers

Open jswiderski opened this issue 3 years ago • 3 comments

📝 Provide detailed reproduction steps (if any)

Based on support ticket np. 75352. Accessibility concern on SVG icons. The svg icons that are used in buttons should have an alt or aria-label or role tag(s) to be AA compliment.

✔️ Expected result

Names of buttons for Type Around feature should be discoverable by screen readers.

❌ Actual result

Names of buttons for Type Around feature are not discoverable by screen readers.

❓ Possible solution

If you have ideas, you can list them here. Otherwise, you can delete this section.

📃 Other details

  • Browser: Any
  • OS: Any
  • First affected CKEditor version: N/A
  • Installed CKEditor plugins: widget

If you'd like to see this fixed sooner, add a 👍 reaction to this post.

jswiderski avatar Jun 20 '22 10:06 jswiderski

Is it possible to increase the priority of this issue? Our application fails the ADA compliance scan, which is mandatory by the US Law. Please provide the estimated date when this issue will be addressed.

zutler avatar Jul 29 '22 15:07 zutler

Hi, I wanted to follow up on this... Could you provide the release version which will include the ADA fixes?

zutler avatar Sep 06 '22 13:09 zutler

I can confirm that labels are missing for the buttons. However, these buttons aren't focusable via keyboard. Probably the better solution would be to:

  • hide these buttons completely from screen readers using [aria-hidden=true],
  • add a description (via [aria-describedby]) to a whole widget with information that the user can type around it by pressing Enter or Shift+Enter.

Comandeer avatar Sep 16 '22 10:09 Comandeer

Just for the record, note that the WidgetTypeAround plugin can be disabled - which turns off the possibility to use type around.

We need to make sure that information about possible type around is removed if the feature is disabled.

mlewand avatar Sep 26 '22 12:09 mlewand