As a followup to #1306760: Accessibility Issues - Add ARIA and #1306752: Accessibility problems with the BUEditor...
Even after the fixes in #1306760, there are still accessibility problems with the 6.x-2.6 version of BUEditor, as noted by dcmouyard in comment #52 on #1306752 (quoting):

--- dcmouyard says:
I did a preliminary accessibility test on the BUEditor for comments. It's keyboard-friendly, which is very nice, but the dialog boxes for adding images, links, and tables are inaccessible.

Here's what I found:

  • You can't tab to the cancel button to exit the dialog box.
  • The form labels need to be <label> elements.
  • There's nothing that tells you which fields are required.
  • There's no error message when you don't fill-in a required field.

Comments

ufku’s picture

ufku commented
Status: Active » Needs review

Pressing ESC key closes the dialog, but i don't know where to put that info.

Here is the list of recent changes i've made:

  1. Made the close button a real button element with a title. It can be focused and clicked using the keyboard.
  2. Added form label elements with "for" attributes.
  3. Added "This field is required." as a title attribute for required fields.

The changes can be tested at the demo site.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

Thanks, that sounds good... I have requested on the other issue and on the Accessibility group on groups.drupal.org that someone test this new version. Hopefully we'll get a response here soon.

ufku’s picture

ufku commented
Status: Needs review » Fixed

Committed the changes to both branches. If, by chance, anyone reviews the changes feel free to make the status active.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.