Make WordPress Core

Opened 9 years ago

Closed 9 years ago

Last modified 8 years ago

#35134 closed defect (bug) (fixed)

Remove title attributes: the Menus screen when no-js

Reported by: afercia's profile afercia Owned by: afercia's profile afercia
Milestone: 4.5 Priority: normal
Severity: normal Version: 4.4
Component: Menus Keywords: has-screenshots, has-patch, title-attribute
Focuses: ui, accessibility Cc:

Description

See related #24766 and all the following tickets about title attributes.

On the "Menus" screen, only when JavaScript is off, some no-JS compatibility links appear and they have title attributes.

I'd suggest to remove the title attributes and use plain text (hidden with screen-reader-text) for the arrows, and just show the "Edit" link. See in the screenshot below, before and after the patch.

https://cldup.com/1Df0x5IZz8.png

Tried to give the "Edit" link just a basic styling, after all it's shown just when JS is off and maybe there's no need for further styling. No visual changes when JavaScript is on.

Attachments (2)

35134.patch (2.7 KB) - added by afercia 9 years ago.
35134.2.patch (2.6 KB) - added by afercia 9 years ago.

Download all attachments as: .zip

Change History (6)

@afercia
9 years ago

#1 @afercia
9 years ago

  • Keywords has-patch added
  • Owner set to afercia
  • Status changed from new to assigned

The proposed patch also removes some mixed spaces/tabs in the CSS.

@afercia
9 years ago

#2 @afercia
9 years ago

Refreshed patch to use aria-label attributes instead of text hidden with .screen-reader-text.

#3 @afercia
9 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 36016:

Accessibility: Remove title attributes and improve accessibility on the "no-js" Menus screen.

When JavaScript is off, the reorder menu item and Edit menu item links now use aria-label
attributes instead of title attributes.

Fixes #35134.

#4 @afercia
8 years ago

  • Keywords title-attribute added
Note: See TracTickets for help on using tickets.