Make WordPress Core

Opened 2 years ago

Last modified 5 weeks ago

#38150 new defect (bug)

Stop using dashicons to show checked state of checkboxes

Reported by: EmpireOfLight Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version:
Component: General Keywords: color-contrast form-controls
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 2 years ago.
Checkboxes use a checkmark dashicon, but they should just use the normal built in browser styling

Download all attachments as: .zip

Change History (9)

2 years 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.

2 years ago

#3 @afercia
2 years ago

  • Focuses ui accessibility added
  • Keywords color-contrast added

Related: #35596

#4 @adamsoucie
23 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.

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

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

#5 follow-up: @EmpireOfLight
23 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
22 months ago

Works for me :)

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

#7 @afercia
22 months ago

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

#8 @afercia
5 weeks ago

  • Keywords form-controls added
  • Milestone changed from Awaiting Review to Future Release

There are a few tickets now asking for form controls redesign, see #44749, #44606, and #35596.

Note: See TracTickets for help on using tickets.