Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'Admin Color Scheme' for Production environments #147

Open
1 task done
rickalee opened this issue Nov 6, 2023 · 2 comments · May be fixed by #165
Open
1 task done

'Admin Color Scheme' for Production environments #147

rickalee opened this issue Nov 6, 2023 · 2 comments · May be fixed by #165
Assignees

Comments

@rickalee
Copy link

rickalee commented Nov 6, 2023

Is your enhancement related to a problem? Please describe.

As a precaution against accidental updates to production environment by 10up engineers and QA team, we should enable a different 'Admin Color Scheme' if 10up user detected and wp_get_environment_type is Production.

https://developer.wordpress.org/reference/functions/wp_get_environment_type/

I suggest we create a custom 'Admin Color Scheme' based on Sunrise core color scheme or 10up brand.

https://wpadmincolors.com/generate

Designs

Screenshot 2023-11-06 at 4 42 54 PM

Describe alternatives you've considered

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@darylldoyle
Copy link
Contributor

@rickalee I'd love to hear your thoughts on including it in a less intrusive way. I'm always concerned about changing colour schemes for a11y reasons. I agree, this could be a massive help though.

I really like the approach taken by https://wordpress.org/plugins/display-environment-type/, what're your thoughts on that?

@fabiankaegy
Copy link
Member

@darylldoyle I really like the approach taken by that plugin! :)

@psorensen psorensen self-assigned this Jul 25, 2024
@psorensen psorensen linked a pull request Jul 25, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants