[wp-trac] [WordPress Trac] #26533: Main navigation and admin bar menu icon colors are inconsistent

WordPress Trac noreply at wordpress.org
Mon Mar 3 16:00:35 UTC 2014


#26533: Main navigation and admin bar menu icon colors are inconsistent
----------------------------+------------------------------
 Reporter:  kovshenin       |       Owner:
     Type:  defect (bug)    |      Status:  new
 Priority:  low             |   Milestone:  Awaiting Review
Component:  Administration  |     Version:  3.8
 Severity:  minor           |  Resolution:
 Keywords:  ui-feedback     |     Focuses:  ui
----------------------------+------------------------------
Changes (by kovshenin):

 * keywords:  reporter-feedback ui-feedback => ui-feedback


Comment:

 > As far as I can see, all the default color schemes always display the
 icon and link text in the same color

 Mostly or close, but not on hover. On hover:

 * Light uses gray for dashboard menu, blue for the admin tooblar
 * Coffee uses white/brown
 * Ectoplasm: white/green
 * Midnight: white/red
 * Ocean: white/green

 Here's some [https://gist.github.com/kovshenin/9327833 sample code] that
 creates a faux menu item with a W logo generated with SVG via the
 background-image property. If you test in on the default color scheme, or
 ones with consistent color behavior across admin toolbar and admin menu,
 everything looks fine.

 But if you switch to one of the color schemes mentioned above, you'll see
 that the SVG painter will repaint the hover icon to the hover color used
 in the dashboard menu, not the toolbar. Hope this makes sense :)

 I guess we should either change the color schemes to be consistent between
 the admin bar and the main menu, or introduce a new js property for the
 admin bar hover color, maybe make it optional for back-compat. I'm leaning
 towards the latter.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/26533#comment:5>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list