Make WordPress Core

Opened 12 years ago

Closed 11 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 12 years ago.
Updates colors that represent gray admin color scheme to the admin refresh

Download all attachments as: .zip

Change History (7)

#1 @dd32
12 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
12 years ago

  • Keywords ui-feedback added

@chexee
12 years ago

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

#3 @chexee
12 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
12 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
11 years ago

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

Patch fine with me.

#6 @ryan
11 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.