WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 3 years ago

#17818 closed defect (bug) (fixed)

Wrong Admin color scheme definition

Reported by: thomask Owned by: 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 4 years ago.
Updates colors that represent gray admin color scheme to the admin refresh

Download all attachments as: .zip

Change History (7)

comment:1 @dd324 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.

comment:2 @SergeyBiryukov4 years ago

  • Keywords ui-feedback added

@chexee4 years ago

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

comment:3 @chexee4 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

comment:4 @jane4 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.

comment:5 @jane4 years ago

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

Patch fine with me.

comment:6 @ryan3 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.