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.

Sign up for GitHub

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

Jump to bottom

[Security Solution] [EUI visual refresh] Apply severity colors #206033

Closed
angorayc opened this issue Jan 9, 2025 · 1 comment · Fixed by #206276
Closed

[Security Solution] [EUI visual refresh] Apply severity colors #206033

angorayc opened this issue Jan 9, 2025 · 1 comment · Fixed by #206276
Labels
EUI Visual Refresh Team:Threat Hunting:Explore

Comments

@angorayc
Copy link
Contributor

angorayc commented Jan 9, 2025 •
edited
Loading

Severity colors needed to be replace according to the following mapping (ref):

// Amsterdam 
(euiTheme.flags.hasVisColorAdjustment)
#54B399 -> euiTheme.colors.vis.euiColorVis0
#D6BF57 -> euiTheme.colors.vis.euiColorVis5
#DA8B45 -> euiTheme.colors.vis.euiColorVis7
#E7664C -> euiTheme.colors.vis.euiColorVis9 

// Borealis - suggestion (feel free to involve a designer)
Low -> euiTheme.colors.vis.euiColorSuccess0 
Moderate -> euiTheme.colors.vis.euiColorSeverity7
High -> euiTheme.colors.vis. euiColorSeverity10
Critical -> euiTheme.colors.vis.euiColorSeverity14

Update: ⚠️ As the final decision for severity color is still pending, the temporary colors are the hard coded color hex:
#203387

Color token Amsterdam Borealis
Critical euiThemeVars.euiColorDanger #E7664C
High euiThemeVars.euiColorVis9_behindText #DA8B45
Meduiml euiThemeVars.euiColorVis5_behindText #D6BF57
Low euiThemeVars.euiColorVis0 #54B399

Relevant implementation:
Cloud Security: https://github.com/elastic/kibana/pull/205136/files#diff-bf3f045d305b5cdcd6ed8b42d1b94a051ec8f8b55270fbd85aed96f93adeb351R35
Entity analytics: https://github.com/elastic/kibana/pull/204007/files

Discussion:
https://elastic.slack.com/archives/C0851RDCWPP/p1737638062361149?thread_ts=1736954935.896329&cid=C0851RDCWPP
@angorayc angorayc mentioned this issue Jan 9, 2025
Closed
@botelastic botelastic bot added the needs-team Issues missing a team label label Jan 9, 2025
@angorayc angorayc added EUI Visual Refresh and removed needs-team Issues missing a team label labels Jan 9, 2025
@botelastic botelastic bot added the needs-team Issues missing a team label label Jan 9, 2025
@angorayc angorayc added Team:Threat Hunting:Explore and removed needs-team Issues missing a team label labels Jan 9, 2025
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-explore (Team:Threat Hunting:Explore)

@angorayc angorayc linked a pull request Jan 13, 2025 that will close this issue
[SecuritySolution] Update severity colors for Borealis theme #206276
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
EUI Visual Refresh Team:Threat Hunting:Explore
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[SecuritySolution] Update severity colors for Borealis theme angorayc/kibana
2 participants
@angorayc @elasticmachine