Make WordPress Core

Opened 22 months ago

Last modified 15 months ago

#38150 new defect (bug)

Stop using dashicons to show checked state of checkboxes

Reported by: EmpireOfLight Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version:
Component: General Keywords: color-contrast
Focuses: ui, accessibility Cc:


Checkboxes should use the default system to indicate they are checked. (<input type="checkbox" checked> uses a dashicon check icon instead of the system's default check ui.

Attachments (1)

pasted_image_at_2016_09_23_03_18_pm.png (81.9 KB) - added by EmpireOfLight 22 months ago.
Checkboxes use a checkmark dashicon, but they should just use the normal built in browser styling

Download all attachments as: .zip

Change History (8)

22 months ago

Checkboxes use a checkmark dashicon, but they should just use the normal built in browser styling

This ticket was mentioned in Slack in #design by empireoflight. View the logs.

17 months ago

#3 @afercia
17 months ago

  • Focuses ui accessibility added
  • Keywords color-contrast added

Related: #35596

#4 @adamsoucie
15 months ago

Is the issue here that the browser's default is being replaced, or that the font color of the dashicon checkmark doesn't meet AA standards? The contrast ratio is 3.78, so if we're just worried about contrast here, darken the blue and we're done.

No need to decouple the dashicon system's that has been built on top of the browser default if all we have to do is improve the color a bit.

Update: [WordPress Blue, 0073AA](http://codepen.io/hugobaeta/full/RNOzoV/) fulfills our color contrast goal without swapping out the SVG implementation.

Last edited 15 months ago by adamsoucie (previous) (diff)

#5 follow-up: @EmpireOfLight
15 months ago

The issue arose when testing svg replacement of the dashicons font in the admin, the checkbox and radios broke and would need to be re-coded (see https://make.wordpress.org/core/2016/10/23/converting-dashicons-to-svg/) . Rather than adding those to the list of fixes, it makes more sense to leave form rendering/chrome to the browser. The admin recently switched to system fonts, and similarly, removing the extra layer of styling might even be a performance boost.

#6 in reply to: ↑ 5 @adamsoucie
15 months ago

Works for me :)

@afercia, should we drop this one from the Color Contrast project as a result?

#7 @afercia
15 months ago

@adamsoucie if it gets merged, sure :) See #35596.

Note: See TracTickets for help on using tickets.