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
[Security Solution] Fix color coding of severities is different at Entity analytics #156383
Conversation
Pinging @elastic/security-threat-hunting (Team:Threat Hunting) |
Pinging @elastic/security-solution (Team: SecuritySolution) |
@@ -14,13 +14,14 @@ import { euiLightVars } from '@kbn/ui-theme'; | |||
|
|||
import { WithHoverActions } from '../../../../../common/components/with_hover_actions'; | |||
import { RiskSeverity } from '../../../../../../common/search_strategy'; | |||
import { SEVERITY_COLOR } from '../../../../../overview/components/detection_response/utils'; | |||
|
|||
export const RISK_SEVERITY_COLOUR: { [k in RiskSeverity]: string } = { | |||
[RiskSeverity.unknown]: euiLightVars.euiColorMediumShade, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have a SEVERITY_COLOR
for this one? Maybe better just to be consistent here and add this one too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Have a minor comment regarding the consistency for the colors source.
f950b93
to
f856f44
Compare
💛 Build succeeded, but was flaky
Failed CI StepsTest Failures
Metrics [docs]Async chunks
Unknown metric groupsESLint disabled line counts
Total ESLint disabled count
History
To update your PR or re-run it, just comment with: cc @machadoum |
…tity analytics (elastic#156383) issue: elastic#143043 ## Summary Fix color coding of severities is different at Entity Analytics by using the same colors of alert severity. **Before:** ![195068297-4893ff3e-184a-4deb-b551-28d55f4994fd](https://user-images.githubusercontent.com/1490444/235682183-272d0739-7a65-42d4-b1ef-83751cf57ab0.jpg) **After:** ![Screenshot 2023-05-02 at 14 40 23](https://user-images.githubusercontent.com/1490444/235678297-f4a05f1c-95f6-4895-a7b1-fd6645e55f7b.png) ![Screenshot 2023-05-02 at 14 40 03](https://user-images.githubusercontent.com/1490444/235678383-0eefe697-5669-496d-b0b8-44433f519754.png) ![Screenshot 2023-05-02 at 14 37 45](https://user-images.githubusercontent.com/1490444/235678609-257b8eb1-8b86-4064-8fbf-0c0b188174ae.png) ![Screenshot 2023-05-02 at 15 32 22](https://user-images.githubusercontent.com/1490444/235681736-609b484c-bd08-4379-b76b-6a47c3a016ba.png) (cherry picked from commit 7f68428)
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
… at Entity analytics (#156383) (#156508) # Backport This will backport the following commits from `main` to `8.8`: - [[Security Solution] Fix color coding of severities is different at Entity analytics (#156383)](#156383) <!--- Backport version: 8.9.7 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Pablo Machado","email":"pablo.nevesmachado@elastic.co"},"sourceCommit":{"committedDate":"2023-05-03T10:46:19Z","message":"[Security Solution] Fix color coding of severities is different at Entity analytics (#156383)\n\nissue: #143043 Summary\r\n\r\nFix color coding of severities is different at Entity Analytics by using\r\nthe same colors of alert severity.\r\n\r\n**Before:**\r\n\r\n![195068297-4893ff3e-184a-4deb-b551-28d55f4994fd](https://user-images.githubusercontent.com/1490444/235682183-272d0739-7a65-42d4-b1ef-83751cf57ab0.jpg)\r\n\r\n\r\n\r\n\r\n**After:**\r\n\r\n![Screenshot 2023-05-02 at 14 40\r\n23](https://user-images.githubusercontent.com/1490444/235678297-f4a05f1c-95f6-4895-a7b1-fd6645e55f7b.png)\r\n![Screenshot 2023-05-02 at 14 40\r\n03](https://user-images.githubusercontent.com/1490444/235678383-0eefe697-5669-496d-b0b8-44433f519754.png)\r\n![Screenshot 2023-05-02 at 14 37\r\n45](https://user-images.githubusercontent.com/1490444/235678609-257b8eb1-8b86-4064-8fbf-0c0b188174ae.png)\r\n\r\n![Screenshot 2023-05-02 at 15 32\r\n22](https://user-images.githubusercontent.com/1490444/235681736-609b484c-bd08-4379-b76b-6a47c3a016ba.png)","sha":"7f68428e3457b1720027ee88b3deb722b0151cbf","branchLabelMapping":{"^v8.9.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["bug","release_note:fix","Team:Threat Hunting","Team: SecuritySolution","Team:Threat Hunting:Explore","v8.8.0","v8.9.0"],"number":156383,"url":"#156383 Solution] Fix color coding of severities is different at Entity analytics (#156383)\n\nissue: #143043 Summary\r\n\r\nFix color coding of severities is different at Entity Analytics by using\r\nthe same colors of alert severity.\r\n\r\n**Before:**\r\n\r\n![195068297-4893ff3e-184a-4deb-b551-28d55f4994fd](https://user-images.githubusercontent.com/1490444/235682183-272d0739-7a65-42d4-b1ef-83751cf57ab0.jpg)\r\n\r\n\r\n\r\n\r\n**After:**\r\n\r\n![Screenshot 2023-05-02 at 14 40\r\n23](https://user-images.githubusercontent.com/1490444/235678297-f4a05f1c-95f6-4895-a7b1-fd6645e55f7b.png)\r\n![Screenshot 2023-05-02 at 14 40\r\n03](https://user-images.githubusercontent.com/1490444/235678383-0eefe697-5669-496d-b0b8-44433f519754.png)\r\n![Screenshot 2023-05-02 at 14 37\r\n45](https://user-images.githubusercontent.com/1490444/235678609-257b8eb1-8b86-4064-8fbf-0c0b188174ae.png)\r\n\r\n![Screenshot 2023-05-02 at 15 32\r\n22](https://user-images.githubusercontent.com/1490444/235681736-609b484c-bd08-4379-b76b-6a47c3a016ba.png)","sha":"7f68428e3457b1720027ee88b3deb722b0151cbf"}},"sourceBranch":"main","suggestedTargetBranches":["8.8"],"targetPullRequestStates":[{"branch":"8.8","label":"v8.8.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.9.0","labelRegex":"^v8.9.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/156383","number":156383,"mergeCommit":{"message":"[Security Solution] Fix color coding of severities is different at Entity analytics (#156383)\n\nissue: #143043 Summary\r\n\r\nFix color coding of severities is different at Entity Analytics by using\r\nthe same colors of alert severity.\r\n\r\n**Before:**\r\n\r\n![195068297-4893ff3e-184a-4deb-b551-28d55f4994fd](https://user-images.githubusercontent.com/1490444/235682183-272d0739-7a65-42d4-b1ef-83751cf57ab0.jpg)\r\n\r\n\r\n\r\n\r\n**After:**\r\n\r\n![Screenshot 2023-05-02 at 14 40\r\n23](https://user-images.githubusercontent.com/1490444/235678297-f4a05f1c-95f6-4895-a7b1-fd6645e55f7b.png)\r\n![Screenshot 2023-05-02 at 14 40\r\n03](https://user-images.githubusercontent.com/1490444/235678383-0eefe697-5669-496d-b0b8-44433f519754.png)\r\n![Screenshot 2023-05-02 at 14 37\r\n45](https://user-images.githubusercontent.com/1490444/235678609-257b8eb1-8b86-4064-8fbf-0c0b188174ae.png)\r\n\r\n![Screenshot 2023-05-02 at 15 32\r\n22](https://user-images.githubusercontent.com/1490444/235681736-609b484c-bd08-4379-b76b-6a47c3a016ba.png)","sha":"7f68428e3457b1720027ee88b3deb722b0151cbf"}}]}] BACKPORT--> Co-authored-by: Pablo Machado <pablo.nevesmachado@elastic.co>
issue: #143043
Summary
Fix color coding of severities is different at Entity Analytics by using the same colors of alert severity.
Before:
After: