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:

Old New
dashicons-admin-sitedashicons-admin-network
dashicons-sitedashicons-admin-site
dashicons-gaugedashicons-admin-dashboard
dashicons-admin-dashboarddashicons-admin-home
dashicons-camera2dashicons-camera
dashicons-imgedit-undodashicons-undo
dashicons-imgedit-redodashicons-redo
dashicons-xitdashicons-dismiss
dashicons-arrow-updashicons-upload
dashicons-arrow-downdashicons-download
dashicons-networkdashicons-networking
dashicons-jobs-developersdashicons-hammer
dashicons-jobs-designersdashicons-art
dashicons-jobs-migrationdashicons-migrate
dashicons-jobs-performancedashicons-performance

Also:

  • 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.

Last edited 11 years ago by DrewAPicture (previous) (diff)

#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.