WordPress.org

Make WordPress Core

Opened 11 months ago

Last modified 4 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:

Description

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 11 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)

@EmpireOfLight
11 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.


6 months ago

#3 @afercia
6 months ago

  • Focuses ui accessibility added
  • Keywords color-contrast added

Related: #35596

#4 @adamsoucie
4 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 4 months ago by adamsoucie (previous) (diff)

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

Works for me :)

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

#7 @afercia
4 months ago

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

Note: See TracTickets for help on using tickets.