[wp-trac] [WordPress Trac] #19878: Allow for a Custom Screen Icon

WordPress Trac wp-trac at lists.automattic.com
Mon Jan 23 16:31:23 UTC 2012


#19878: Allow for a Custom Screen Icon
-----------------------------+------------------------------
 Reporter:  wpsmith          |       Owner:
     Type:  enhancement      |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  General          |     Version:
 Severity:  normal           |  Resolution:
 Keywords:  has-patch close  |
-----------------------------+------------------------------
Changes (by TobiasBg):

 * keywords:  has-patch => has-patch close


Comment:

 I don't think that this filter is actually necessary.
 When you pass a string to {{{screen_icon()}}}, like
 {{{
 screen_icon( 'my-plugin' );
 }}}
 the resulting {{{<div>}}} will have the ID {{{#icon-my-plugin}}}. You'll
 just need to add something like
 {{{
 #icon-my-plugin {
         background: transparent url( 'plugin-icon.png' ) no-repeat;
 }
 }}}
 to the CSS file that your (or most) plugins load on their admin screens
 anyway. I don't see the need for adding a filter to add extra inline CSS
 styling.

 Suggesting wont-fix.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/19878#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list