Skip to content

Commit c2745fe

Browse files
authored
chore: improved status colors (podman-desktop#8236)
Issue podman-desktop#6925 was reopened to tweak the provider status and card carousel colors. The first thing I found was that providers weren't using the correct colors (variables or values) to begin with, so that is covered in issue podman-desktop#8234/PR podman-desktop#8235. Since we are now using consistent/one set of color variables, changing these status colors for providers implies changing these *everywhere*, including the status icons for containers, pods, deployments, etc. That said, I thought the previous colors were a little bright and I prefer these corrected ones in both light and dark mode. Fixes podman-desktop#6925. Signed-off-by: Tim deBoer <[email protected]>
1 parent d0cee19 commit c2745fe

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

packages/main/src/plugin/color-registry.ts

+10-10
Original file line numberDiff line numberDiff line change
@@ -552,7 +552,7 @@ export class ColorRegistry {
552552

553553
this.registerColor(`${ct}card-carousel-card-text`, {
554554
dark: colorPalette.gray[400],
555-
light: colorPalette.purple[900],
555+
light: colorPalette.charcoal[500],
556556
});
557557

558558
this.registerColor(`${ct}card-carousel-nav`, {
@@ -1139,31 +1139,31 @@ export class ColorRegistry {
11391139

11401140
// Podman & Kubernetes
11411141
this.registerColor(`${status}running`, {
1142-
dark: colorPalette.green[400],
1143-
light: colorPalette.green[400],
1142+
dark: colorPalette.green[500],
1143+
light: colorPalette.green[600],
11441144
});
11451145
// Kubernetes only
11461146
this.registerColor(`${status}terminated`, {
11471147
dark: colorPalette.red[500],
1148-
light: colorPalette.red[500],
1148+
light: colorPalette.red[700],
11491149
});
11501150
this.registerColor(`${status}waiting`, {
11511151
dark: colorPalette.amber[600],
11521152
light: colorPalette.amber[600],
11531153
});
11541154
// Podman only
11551155
this.registerColor(`${status}starting`, {
1156-
dark: colorPalette.green[600],
1156+
dark: colorPalette.green[500],
11571157
light: colorPalette.green[600],
11581158
});
11591159
// Stopped & Exited are the same color / same thing in the eyes of statuses
11601160
this.registerColor(`${status}stopped`, {
1161-
dark: colorPalette.gray[300],
1162-
light: colorPalette.gray[600],
1161+
dark: colorPalette.gray[900],
1162+
light: colorPalette.charcoal[200],
11631163
});
11641164
this.registerColor(`${status}exited`, {
1165-
dark: colorPalette.gray[300],
1166-
light: colorPalette.gray[600],
1165+
dark: colorPalette.gray[900],
1166+
light: colorPalette.charcoal[200],
11671167
});
11681168
this.registerColor(`${status}not-running`, {
11691169
dark: colorPalette.gray[700],
@@ -1185,7 +1185,7 @@ export class ColorRegistry {
11851185
});
11861186
this.registerColor(`${status}dead`, {
11871187
dark: colorPalette.red[500],
1188-
light: colorPalette.red[500],
1188+
light: colorPalette.red[700],
11891189
});
11901190
// If we don't know the status, use gray
11911191
this.registerColor(`${status}unknown`, {

0 commit comments

Comments
 (0)