Opened 11 years ago
Last modified 10 days ago
#21516 new enhancement
Make the entire .check-column the click target for the checkbox
Reported by: |
|
Owned by: | |
---|---|---|---|
Milestone: | 6.3 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Administration | Keywords: | has-ux-feedback dev-feedback has-patch |
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 (3)
Change History (15)
#3
@
10 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.
This ticket was mentioned in Slack in #design by karmatosed. View the logs.
6 years ago
#6
@
6 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?
This ticket was mentioned in Slack in #core by audrasjb. View the logs.
20 months ago
#9
@
20 months 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.
This ticket was mentioned in Slack in #accessibility by joedolson. View the logs.
7 weeks ago
#11
@
7 weeks ago
- Keywords 2nd-opinion removed
- Milestone changed from Future Release to 6.3
We think this could be beneficial for accessibility by providing a larger click target for users. The JS solution isn't really necessary; if we make the label a block-level element filling the table cell & move the screen reader text class to an internal span, then we get this behavior for free.
We may want to add some kind of hover state to provide affordance that the entire cell is clickable, although the mouse cursor will provide a clue there.
#12
@
10 days ago
- Keywords has-patch added; needs-patch needs-refresh removed
21516.3.diff would need some polishing (indentation, etc.). Note that it does not affect the Select All checkboxes on the Updates page because those have a label in a separate cell. The hover style uses the same box-shadow
as the focus style, which might confuse some mouse users after they select a checkbox and hover over another.
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.