WordPress.org

Make WordPress Core

Opened 9 years ago

Last modified 7 days ago

#21516 new enhancement

Make the entire .check-column the click target for the checkbox

Reported by: mitchoyoshitaka Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: Administration Keywords: has-ux-feedback needs-patch dev-feedback needs-refresh 2nd-opinion
Focuses: accessibility Cc:

Description

Sometimes checkboxes are quite small. In various list tables, checkboxes are wrapped in a .check-column. Why not make the entire .check-column clickable, in case people (like me) miss?

Attachments (2)

21516.diff​ (851 bytes) - added by mitchoyoshitaka 9 years ago.
21516.2.diff​ (827 bytes) - added by lessbloat 9 years ago.

Download all attachments as: .zip

Change History (11)

@mitchoyoshitaka
9 years ago

@lessbloat
9 years ago

#1 @lessbloat
9 years ago

Looks good to me. In 21516.2.diff I updated the patch, and made a slight tweak to include this functionality for the "check all" checkboxes in the table header row.

#2 @TacoVerdo
8 years ago

Current patch breaks checking.
Cannot use check all anymore.

#3 @joostdevalk
8 years ago

  • Keywords needs-patch added; has-patch removed

Looked at this with @TacoVerdo, this needs a new patch if we want to fix it that ties into the check all functionality. Right now it doesn't check all when it should and doesn't check the current checkbox when you click it.

#4 @chriscct7
6 years ago

  • Keywords dev-feedback added
  • Severity changed from minor to normal

This ticket was mentioned in ​Slack in #design by karmatosed. ​View the logs.
4 years ago

#6 @karmatosed
4 years ago

  • Keywords has-ux-feedback added; ux-feedback removed

This would be great to have as an enhancement, we discussed this in our weekly design triage and it would be great to have implemented. Could we get a new patch for this?

#7 @Hareesh Pillai
2 months ago

  • Milestone changed from Awaiting Review to 5.9

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs.
7 days ago

#9 @audrasjb
7 days ago

  • Focuses accessibility added
  • Keywords needs-refresh 2nd-opinion added
  • Milestone changed from 5.9 to Future Release

The patch is 9 years old and needs refresh. Adding accessibility focus and 2nd-opinion. I don't think it's safe for 5.9 as it is now. Let's move to future release. Feel free to bring it back to 5.9 if a working patch is ready to go.

Note: See TracTickets for help on using tickets.