Ticket #7399 (closed enhancement: fixed)

Opened 4 months ago

Last modified 4 months ago

Filter to disable admin color scheme selection

Reported by: axelseaa Assigned to: 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

(follow-up: ↓ 2 ) 07/24/08 18:04:25 changed by westi

  • keywords set to reporter-feedback.
  • owner changed from anonymous to westi.
  • version set to 2.6.
  • status changed from new to assigned.

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.

(in reply to: ↑ 1 ) 07/24/08 18:16:26 changed by axelseaa

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.

07/24/08 20:57:57 changed by westi

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.

07/24/08 21:55:45 changed by axelseaa

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

(follow-up: ↓ 7 ) 07/26/08 07:54:02 changed by westi

  • status changed from assigned to closed.
  • resolution set to fixed.

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

07/26/08 07:58:01 changed by westi

  • milestone changed from 2.6.1 to 2.7.

(in reply to: ↑ 5 ) 07/26/08 08:01:11 changed by DD32

  • status changed from closed to reopened.
  • resolution deleted.

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.

07/26/08 08:03:15 changed by westi

bah. html. bah

07/26/08 08:05:30 changed by westi

  • status changed from reopened to closed.
  • resolution set to fixed.

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