Make WordPress Core

Opened 13 years ago

Closed 13 years ago

#17818 closed defect (bug) (fixed)

Wrong Admin color scheme definition

Reported by: thomask's profile thomask Owned by: ryan's profile ryan
Milestone: 3.4 Priority: normal
Severity: minor Version: 3.2
Component: UI Keywords: has-patch
Focuses: Cc:

Description (last modified by chexee)


I have looked in wp_inc/general-template.php

where is defined the admin scheme:

wp_admin_css_color( 'fresh', __( 'Gray' ), admin_url( 'css/colors-fresh.css' ), array( '#7c7976', '#c6c6c6', '#e0e0e0', '#f1f1f1' ) );

the array define colors, which are then used in the theme so it helps identify the look of the scheme.

The problem is that current Gray (btw why it is named Gray and then everything else is "fresh"?) do not use 3 of that 4 colors (only #f1f1f1 is there).

of course it is not a "problem" - it just does not make sense

Attachments (1)

17818.patch (626 bytes) - added by chexee 13 years ago.
Updates colors that represent gray admin color scheme to the admin refresh

Download all attachments as: .zip

Change History (7)

#1 @dd32
13 years ago

  • Description modified (diff)

If anyone wants a visual overview of the colours used in the stylesheet, I just made this quick post: http://dd32.id.au/2011/06/17/wordpress-3-2-admin-colour-palette/, someone might want to cherry pick colours from there which suit better than the existing ones.

#2 @SergeyBiryukov
13 years ago

  • Keywords ui-feedback added

@chexee
13 years ago

Updates colors that represent gray admin color scheme to the admin refresh

#3 @chexee
13 years ago

  • Description modified (diff)
  • Keywords has-patch added
  • Milestone changed from Awaiting Review to Future Release

Chose a few colors based on dd32's post:

http://chx.mx//4z/p4miptv90csww.png

#4 @jane
13 years ago

They used to be "Fresh" and "Classic" in the olden days before we went for the more descriptive UI labels of Gray and Blue.

#5 @jane
13 years ago

  • Keywords ui-feedback removed
  • Milestone changed from Future Release to 3.4

Patch fine with me.

#6 @ryan
13 years ago

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

In [19851]:

Update color scheme definition for Gray Theme. Props chexee. fixes #17818

Note: See TracTickets for help on using tickets.