WordPress.org

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

#7399 closed enhancement (fixed)

Filter to disable admin color scheme selection

Reported by: axelseaa Owned by: westi
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.6
Component: General Keywords: reporter-feedback
Focuses: Cc:

Description

This would again mainly be for MU - but we have a custom admin css, and the Admin color scheme box doesn't do the users any good. Can we add some sort of filter to disable it? Thanks again!

Change History (9)

#1 follow-up: @westi
12 years ago

  • Keywords reporter-feedback added
  • Owner changed from anonymous to westi
  • Status changed from new to assigned
  • Version set to 2.6

What do you mean by disable?

What sort of filter do you want here.

I think it should already be possible to have a single custom admin color scheme with the current code.

#2 in reply to: ↑ 1 @axelseaa
12 years ago

Replying to westi:

What do you mean by disable?

What sort of filter do you want here.

I think it should already be possible to have a single custom admin color scheme with the current code.

Ideally I would like a filter to hide that box all together. What I have done in the paste is to add my own custom css to the admin_head action.

#3 @westi
12 years ago

I think the best solution here would be for the code on the profile page to only show the options if there is more than one theme and for you to remove the built in themes using a plugin to modify the globals (or even better we could add a wp_admin_remove_css function to allow them to be unregistered.

#4 @axelseaa
12 years ago

That would work as well, whatever you guys think is the best solution. All my goal is just to make the box disappear!

#5 follow-up: @westi
12 years ago

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [8461]) Only show the admin colour scheme picker if there is more than one theme to pick from. Fixes #7399.

#6 @westi
12 years ago

  • Milestone changed from 2.6.1 to 2.7

#7 in reply to: ↑ 5 @DD32
12 years ago

  • Resolution fixed deleted
  • Status changed from closed to reopened

Replying to westi:

(In [8461]) Only show the admin colour scheme picker if there is more than one theme to pick from. Fixes #7399.

The endif is after the </table> tag, Yet the colour picker is mearly a row; Lines 199 & 200 need switching around.

#8 @westi
12 years ago

bah. html. bah

#9 @westi
12 years ago

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [8462]) Move the endif inside the table so [8461] doesn't break the html. Fixes #7399 props DD32.

Note: See TracTickets for help on using tickets.