[wp-trac] [WordPress Trac] #40678: Editing menus in WP admin for blind people

WordPress Trac noreply at wordpress.org
Sun Feb 18 14:34:04 UTC 2018


#40678: Editing menus in WP admin for blind people
--------------------------+----------------------------
 Reporter:  juliemoynat   |       Owner:
     Type:  defect (bug)  |      Status:  new
 Priority:  normal        |   Milestone:  5.0
Component:  Menus         |     Version:
 Severity:  normal        |  Resolution:
 Keywords:  has-patch     |     Focuses:  accessibility
--------------------------+----------------------------

Comment (by audrasjb):

 Thanks @juliemoynat !

 > * '''add a title before each block:''' I thought more of adding a "h2"
 tag (instead of a "span") because people can navigate through titles with
 their screen readers. I know that sighted persons who use screen readers
 don't like to have hidden titles but that can be a real help for blind
 persons, in my opinion.
 Fixed in `40678.7.diff`
 > * '''add a screen reader text to “Add to menu” button:''' good job. You
 find the right words ;-)
 Great ;)
 > * '''add a screen reader text to links in the right column (just like in
 the left one):''' the text added is not at the right place (in the diff,
 it is in a "div" that can not receive focus). For this text being
 readable, we should add it in the link that have "item-edit" class. But...
 now, I see that this link has an aria-label attribute. (For information,
 an "aria-label" text overrides the content inside tags in a screen
 reader.) This attribute is suddenly modified with JavaScript... So this is
 more a bug to fix in JS.
 This one is quite tricky to implement. I'm still working on it.
 > * '''add a screen reader text after the 2 first tabs:''' I'm looking for
 the "add-edit-menu-action" class in my browser web developer tool to see
 where the text is added but I can't find it. Is the text adding to the
 right place? I think this text could be placed just before the "div
 .manage-menus" element in a "p" tag (because it's a paragraph ;-) ).
 Fixed in `40678.7.diff`

 @afercia Theorical question: what is the process for partial fixes? Is it
 okay to commit some fixes but let the ticket open?

 Cheers,
 Jb

--
Ticket URL: <https://core.trac.wordpress.org/ticket/40678#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list