Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#26227 closed defect (bug) (fixed)

Dashicons: Include the easy-access classes for plugins

Reported by: dd32's profile dd32 Owned by: nacin's profile nacin
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Graphic Design Keywords: has-patch 2nd-opinion
Focuses: Cc:

Description

Currently the version of Dashicons in core doesn't include the easy-access/easy-use CSS classes such as .dashicons and .dashicon-wordpress, which forces themes to use the glyphs directly by specifying the fonts and content:'\f155';.

Instead, we should include the full set of icon classes as visible within the Dashicons CSS: https://github.com/melchoyce/dashicons/blob/master/css/dashicons.css

Attachments (2)

26227.diff (10.1 KB) - added by dd32 11 years ago.
26227.2.diff (54.0 KB) - added by melchoyce 11 years ago.

Download all attachments as: .zip

Change History (8)

#1 @melchoyce
11 years ago

  • Cc melissachoyce@… added

I kind of guessed on some of the class names — I'd appreciate a second look to make sure everything I've named them makes sense.

@dd32
11 years ago

#2 @dd32
11 years ago

26227.diff is the extra rules.

The class names are visible within there, and also on the demo site here: http://melchoyce.github.io/dashicons/

I personally haven't seen any I disagree with, but it'd be helpful if others could comb through it as well.

#3 @dd32
11 years ago

  • Keywords has-patch 2nd-opinion added

#4 @nacin
11 years ago

  • Owner set to nacin
  • Resolution set to fixed
  • Status changed from new to closed

In 26547:

Add dashicons CSS classes for easy access by plugins and themes.

props dd32.
fixes #26227.

@melchoyce
11 years ago

#5 @melchoyce
11 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

In 26227.2.diff, I updated the following icon class names:

  • dashicons-admin-site --> dashicons-admin-network
  • dashicons-site --> dashicons-admin-site
  • dashicons-gauge --> dashicons-admin-dashboard
  • dashicons-admin-dashboard --> dashicons-admin-home
  • dashicons-camera2 --> dashicons-camera
  • dashicons-imgedit-undo --> dashicons-undo
  • dashicons-imgedit-redo --> dashicons-redo
  • dashicons-xit --> dashicons-dismiss
  • dashicons-arrow-up --> dashicons-upload
  • dashicons-arrow-down --> dashicons-download
  • dashicons-network --> dashicons-networking
  • dashicons-jobs-developers --> dashicons-hammer
  • dashicons-jobs-designers --> dashicons-art
  • dashicons-jobs-migration --> dashicons-migrate
  • dashicons-jobs-performance --> dashicons-performance
  • imgedit --> image, write out full name (rleft --> rotate-left)
  • graphs to --> chart-bar, chart-line, chart-area, chart-pie
  • change everything with 1/2/3 to alt, alt-2
  • change all arr to arrow
  • remove twitter-alt b/c it's not correct branding

I also updated the data-uri for the font based on the latest version of dashicons.

Version 0, edited 11 years ago by melchoyce (next)

#6 @nacin
11 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

In 26752:

Updates to Dashicons classes.

props melchoyce.
fixes #26227.

Note: See TracTickets for help on using tickets.