Opened 5 years ago

Closed 5 years ago

#7399 closed enhancement (fixed)

Filter to disable admin color scheme selection

Reported by: axelseaa Owned by: westi
Priority: normal Milestone: 2.7
Component: General Version: 2.6
Severity: normal Keywords: reporter-feedback
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)

comment:1 follow-up: ↓ 2   westi5 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.

comment:2 in reply to: ↑ 1   axelseaa5 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.

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.

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

comment:5 follow-up: ↓ 7   westi5 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.

  • Milestone changed from 2.6.1 to 2.7

comment:7 in reply to: ↑ 5   DD325 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.

bah. html. bah

  • 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.