WordPress.org
Get WordPress

Make WordPress Core

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#25967 closed defect (bug) (fixed)

Appearance Themes: current theme highlight color should match the color scheme

Reported by: matveb Owned by: iammattthomas
Priority: normal Milestone: 3.8
Component: Customize Version: 3.8
Severity: normal Keywords: has-patch
Cc: Focuses: ui

Description

The current theme highlight is currently set to blue, but it should be set by the mp6 color scheme instead.

Attachments (1)

25967.diff​ (430 bytes ) - added by ryelle 13 years ago.

Download all attachments as: .zip

Change History (7)

#1 @SergeyBiryukov
13 years ago

  • Keywords ui-focus added
  • Milestone Awaiting Review → 3.8

@ryelle
13 years ago

#2 @ryelle
13 years ago

25967.diff​ adds the theme-name to the _admin.scss file, which is used to generate the color schemes. It uses the $base-color as background & $text-color as text color (this matches the admin menu in all core schemes).

#3 @melchoyce
13 years ago

  • Keywords has-patch added

#4 @ocean90
13 years ago

.theme-browser .add-new-theme:hover:after and .theme-browser .add-new-theme:hover span:after are needed too.
Shouldn't it be $highlight-color?

Last edited 13 years ago by ocean90 (previous) (diff)

#5 @iammattthomas
13 years ago

  • Owner set to iammattthomas
  • Resolution → fixed
  • Status new → closed

In 26422:

Match current theme background and add theme hover state color to the highlight color in color schemes. Fixes #25967, props ryelle.

#6 @iammattthomas
13 years ago

r26422 is ryelle's patch + the add-new-theme hover state. Only other color in themes is the orange used when an update is available; I'm not sure if that color is used anywhere else in core, but I think it's a separate issue.

Last edited 13 years ago by iammattthomas (previous) (diff)
Note: See TracTickets for help on using tickets.