Skip to content

Conversation

@mhokanson
Copy link

Update the default color for the Administrator window to further differentiate OIE from Mirth Connect.

Issue 177

Update the default color for the Administrator window to further differentiate OIE from Mirth Connect.

Issue 177

Signed-off-by: mhokanson <mhokanson@gmail.com>
@tonygermano tonygermano linked an issue Sep 5, 2025 that may be closed by this pull request
@pacmano1
Copy link
Contributor

pacmano1 commented Sep 5, 2025

I get the intention here - but as I mentioned on the issue - I do not like this PR. The new default color may be someone's color used to represent something. e.g. red for Prod, etc.

A muted pattern would be better or some other visual indicator.

@tonygermano tonygermano changed the title Update ServerSettings.java Update default color for Administrator Sep 5, 2025
@mhokanson
Copy link
Author

Here's a side-by-side comparison of the change.

Again, this only affects the out-of-the-box default color and won't affect custom colors already set and saved (example: Production in "Warning Red").

image

@kpalang kpalang requested review from a team, jonbartels, kayyagari, kpalang and tonygermano and removed request for a team September 8, 2025 15:29
@kryskool
Copy link

kryskool commented Sep 9, 2025

light blue is a good compromise (less aggressive than orange)

Copy link
Member

@tonygermano tonygermano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should give this one another week or so to sit to see if it gets any additional feedback from the community.

Copy link
Contributor

@pacmano1 pacmano1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not a fan of the PR, but seems like most people think it is a good idea.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[IDEA] Pimp my OIE Administrator

6 participants