#31443 closed defect (bug) (fixed)
Press This: Mega checkboxes in IE and Firefox
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | normal | Version: | 4.2 |
Component: | Press This | Keywords: | has-patch |
Focuses: | ui | Cc: |
Description
This is due to an attempt at focus styles on the categories list. The best thing to do would be to change the categories list markup to li
elements.
Pros:
- full style control
- A11y win (list tells how many items are in it)
- removal of super hacky CSS targeting browsers
Working example: http://test.cita.uiuc.edu/aria/checkbox/checkbox1.php
Attachments (4)
Change History (17)
This ticket was mentioned in Slack in #core by michaelarestad. View the logs.
9 years ago
#5
follow-up:
↓ 6
@
9 years ago
Started styling this. I'm attaching what I've done so far, but as you'll see, there is some serious padding issues due to the nesting. Any way we could change the layout a little? Something like this would work:
<ul> <li> <div aria and checkbox stuff>Category name</div> <ul> <li> <div aria and checkbox stuff>Category name</div> </li> </ul> </li> </ul>
#8
@
9 years ago
- Owner set to azaozz
- Resolution set to fixed
- Status changed from new to closed
In 31765:
#11
@
9 years ago
In addition to comment 10 by @azaozz: Screen reader test results for role="application"
are at https://make.wordpress.org/accessibility/2015/03/26/usertest-extra-small-test-about-roleapplication-in-press-this/
Note: See
TracTickets for help on using
tickets.
In 31443.patch:
Needs styling.