Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#26145 closed defect (bug) (fixed)

Row actions are not keyboard accessible

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Accessibility Keywords: has-patch commit
Focuses: ui Cc:


Background: #21334

Tabbing to the items in the lists no longer triggers appearance of Quick Edit links.

Fixed for 3.7 in [25595], which depended on [25592].

Re-broken in [26072], which essentially reverted [25592].

Attachments (2)

26145.patch (1.3 KB) - added by SergeyBiryukov 11 years ago.
26145.2.patch (1.3 KB) - added by SergeyBiryukov 11 years ago.

Download all attachments as: .zip

Change History (9)

#1 @SergeyBiryukov
11 years ago

  • Keywords commit added

26145.patch re-introduces [25592].

26145.2.patch also removes lines 132 and 133 in colors-fresh.css, which seem redundant.

#2 follow-up: @dd32
11 years ago

How are you testing this?

From what I can see, under both a clean 3.7 and trunk, tabbing to the fields does not work on the plugins page, or the Posts page.
Under 3.7 the Plugins links are forced to be displayed (but Posts are not) and this patch matches that style.. but it doesn't appear to actually make it accessible the same way that [25595] was trying to do

#3 in reply to: ↑ 2 @SergeyBiryukov
11 years ago

Replying to dd32:

How are you testing this?

  1. Go to the Posts screen.
  2. Click the checkbox on the left of any post and press Tab.

In 3.7, this will reveal the action links for that post. In trunk, nothing happens.

The JS introduced in [25595] still adds the .visible class, but the proper styling is missing.

.row-actions-visible class no longer exists in core since [25592], but the CSS part of that changeset was reverted during the merge in [26072].

Last edited 11 years ago by SergeyBiryukov (previous) (diff)

#4 @dd32
11 years ago

Ahah, got it now.
With the patch the Posts screen now works as explained (Row actions show when tab from the checkbox, but hidden otherwise), however the plugins page always shows the actions, can we make it work like the Posts page too?

#5 @SergeyBiryukov
11 years ago

Plugins isn't the only screen that always shows the actions (since [11367], moved to list tables in [15924]), Themes in network admin does too (since [16113]).

We could probably change that, but it sounds like a separate UI/UX change to me that should be done in another ticket.

#6 @SergeyBiryukov
11 years ago

  • Owner set to SergeyBiryukov
  • Resolution set to fixed
  • Status changed from new to closed

In 26324:

Fix keyboard accessibility for row actions in list tables. Restore the CSS part of [25592], accidentally reverted in [26072].

fixes #26145.

#7 @SergeyBiryukov
11 years ago

#26166 was marked as a duplicate.

Note: See TracTickets for help on using tickets.