Skip to content

Commit af79829

Browse files
committed
[One Discover] Revert token change from vis palette (elastic#204054)
## 📓 Summary Related to elastic#202985 This change reverts a suggestion that was applied but that should only be valid for v9. Co-authored-by: Marco Antonio Ghiani <[email protected]> (cherry picked from commit 08da946) # Conflicts: # packages/kbn-discover-utils/src/data_types/logs/utils/get_log_level_color.test.ts # packages/kbn-discover-utils/src/data_types/logs/utils/get_log_level_color.ts
1 parent ac450e9 commit af79829

File tree

2 files changed

+21
-12
lines changed

2 files changed

+21
-12
lines changed

packages/kbn-discover-utils/src/data_types/logs/utils/get_log_level_color.test.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ import { LogLevelCoalescedValue } from './get_log_level_coalesed_value';
1313

1414
const euiTheme = {
1515
colors: {
16-
lightShade: '#ffffff',
16+
mediumShade: '#d3dae6',
1717
},
1818
};
1919

@@ -32,20 +32,20 @@ describe('getLogLevelColor', () => {
3232
'#d6bf57'
3333
);
3434
expect(getLogLevelColor(LogLevelCoalescedValue.error, euiTheme as EuiThemeComputed)).toBe(
35-
'#df9352'
35+
'#e18774'
3636
);
3737
expect(getLogLevelColor(LogLevelCoalescedValue.critical, euiTheme as EuiThemeComputed)).toBe(
38-
'#e7664c'
38+
'#dd7b67'
3939
);
4040
expect(getLogLevelColor(LogLevelCoalescedValue.alert, euiTheme as EuiThemeComputed)).toBe(
41-
'#da5e47'
41+
'#d76f5b'
4242
);
4343
expect(getLogLevelColor(LogLevelCoalescedValue.emergency, euiTheme as EuiThemeComputed)).toBe(
44-
'#cc5642'
44+
'#d2634e'
4545
);
4646
// other
4747
expect(getLogLevelColor(LogLevelCoalescedValue.trace, euiTheme as EuiThemeComputed)).toBe(
48-
'#ffffff'
48+
'#d3dae6'
4949
);
5050
});
5151
});

packages/kbn-discover-utils/src/data_types/logs/utils/get_log_level_color.ts

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,12 @@
77
* License v3.0 only", or the "Server Side Public License, v 1".
88
*/
99

10-
import { EuiThemeComputed, euiPaletteForTemperature, euiPaletteForStatus } from '@elastic/eui';
10+
import {
11+
EuiThemeComputed,
12+
euiPaletteForTemperature,
13+
euiPaletteForStatus,
14+
euiPaletteRed,
15+
} from '@elastic/eui';
1116
import { LogLevelCoalescedValue } from './get_log_level_coalesed_value';
1217

1318
export const getLogLevelColor = (
@@ -16,8 +21,11 @@ export const getLogLevelColor = (
1621
): string | undefined => {
1722
const euiPaletteForTemperature6 = euiPaletteForTemperature(6);
1823
const euiPaletteForStatus9 = euiPaletteForStatus(9);
24+
const euiPaletteRed9 = euiPaletteRed(14);
1925

2026
switch (logLevelCoalescedValue) {
27+
case LogLevelCoalescedValue.trace:
28+
return euiTheme.colors.mediumShade;
2129
case LogLevelCoalescedValue.debug:
2230
return euiPaletteForTemperature6[2]; // lighter, closer to the default color for all other unknown log levels
2331
case LogLevelCoalescedValue.info:
@@ -27,15 +35,16 @@ export const getLogLevelColor = (
2735
case LogLevelCoalescedValue.warning:
2836
return euiPaletteForStatus9[4];
2937
case LogLevelCoalescedValue.error:
30-
return euiPaletteForStatus9[5];
38+
return euiPaletteRed9[9];
3139
case LogLevelCoalescedValue.critical:
32-
return euiPaletteForStatus9[6];
40+
return euiPaletteRed9[10];
3341
case LogLevelCoalescedValue.alert:
34-
return euiPaletteForStatus9[7];
42+
return euiPaletteRed9[11];
3543
case LogLevelCoalescedValue.emergency:
44+
return euiPaletteRed9[12];
3645
case LogLevelCoalescedValue.fatal:
37-
return euiPaletteForStatus9[8];
46+
return euiPaletteRed9[13];
3847
default:
39-
return euiTheme.colors.lightShade;
48+
return euiTheme.colors.mediumShade;
4049
}
4150
};

0 commit comments

Comments
 (0)