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.

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 ] Fixes Timeline Unsaved changes badge color as per EUI recs #207351

Merged
merged 3 commits into from
Jan 21, 2025

Conversation

logeekal
Copy link
Contributor

@logeekal logeekal commented Jan 21, 2025

Summary

Handles

Before After
Light theme
timeline_badge_before_fix.mp4
timeline_badge_color_fix.mp4
Borealis Timeline Minimized image image
Borealis Timeline Maximized ( remains unchanged ) image image

@logeekal logeekal changed the title fix: badge color as per EUI recs [Security Solution ] Fixes Timeline Unsaved changes badge color as per EUI recs Jan 21, 2025
@logeekal logeekal added release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) labels Jan 21, 2025
@logeekal logeekal marked this pull request as ready for review January 21, 2025 13:29
@logeekal logeekal requested a review from a team as a code owner January 21, 2025 13:29
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations)

@logeekal logeekal enabled auto-merge (squash) January 21, 2025 13:41
@logeekal logeekal merged commit 089a91c into elastic:main Jan 21, 2025
8 checks passed
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.x

https://github.com/elastic/kibana/actions/runs/12889745298

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
securitySolution 21.3MB 21.3MB +336.0B

@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.x

Note: Successful backport PRs will be merged automatically after passing CI.

Questions ?

Please refer to the Backport tool documentation

kibanamachine added a commit that referenced this pull request Jan 21, 2025
… badge color as per EUI recs (#207351) (#207380)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[Security Solution ] Fixes Timeline `Unsaved changes` badge
color as per EUI recs
(#207351)](#207351)

<!--- Backport version: 9.4.3 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Jatin
Kathuria","email":"[email protected]"},"sourceCommit":{"committedDate":"2025-01-21T15:02:56Z","message":"[Security
Solution ] Fixes Timeline `Unsaved changes` badge color as per EUI recs
(#207351)\n\n## Summary\n\nHandles\n-
https://github.com/elastic/kibana/issues/205733\n\n||Before|After|\n|---|---|---|\n|Light
theme|
<video\nsrc=\"https://github.com/user-attachments/assets/b8b89b2f-3c4f-40c1-b5b9-88a1044208cf\"\n/>|
<video\nsrc=\"https://github.com/user-attachments/assets/33495309-c672-47c1-9f0d-e26892b62ef6\"\n/>|\n|
Borealis Timeline Minimized
|\n![image](https://github.com/user-attachments/assets/e07d0672-95ad-4e98-9695-c33064f5cc7d)|\n![image](https://github.com/user-attachments/assets/98dfc8d4-27e0-4e6a-82c0-117117c820d4)\n|\n|
Borealis Timeline Maximized ( remains unchanged )
|\n![image](https://github.com/user-attachments/assets/0e685d86-30d5-4fa0-8ed9-87ce98bac8a7)\n|\n![image](https://github.com/user-attachments/assets/978a9269-9180-4f92-96e9-257d0ae475e9)\n|","sha":"089a91c8287da8fc789e2144d778e7daddba44d1","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","backport:prev-minor"],"title":"[Security
Solution ] Fixes Timeline `Unsaved changes` badge color as per EUI
recs","number":207351,"url":"https://github.com/elastic/kibana/pull/207351","mergeCommit":{"message":"[Security
Solution ] Fixes Timeline `Unsaved changes` badge color as per EUI recs
(#207351)\n\n## Summary\n\nHandles\n-
https://github.com/elastic/kibana/issues/205733\n\n||Before|After|\n|---|---|---|\n|Light
theme|
<video\nsrc=\"https://github.com/user-attachments/assets/b8b89b2f-3c4f-40c1-b5b9-88a1044208cf\"\n/>|
<video\nsrc=\"https://github.com/user-attachments/assets/33495309-c672-47c1-9f0d-e26892b62ef6\"\n/>|\n|
Borealis Timeline Minimized
|\n![image](https://github.com/user-attachments/assets/e07d0672-95ad-4e98-9695-c33064f5cc7d)|\n![image](https://github.com/user-attachments/assets/98dfc8d4-27e0-4e6a-82c0-117117c820d4)\n|\n|
Borealis Timeline Maximized ( remains unchanged )
|\n![image](https://github.com/user-attachments/assets/0e685d86-30d5-4fa0-8ed9-87ce98bac8a7)\n|\n![image](https://github.com/user-attachments/assets/978a9269-9180-4f92-96e9-257d0ae475e9)\n|","sha":"089a91c8287da8fc789e2144d778e7daddba44d1"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/207351","number":207351,"mergeCommit":{"message":"[Security
Solution ] Fixes Timeline `Unsaved changes` badge color as per EUI recs
(#207351)\n\n## Summary\n\nHandles\n-
https://github.com/elastic/kibana/issues/205733\n\n||Before|After|\n|---|---|---|\n|Light
theme|
<video\nsrc=\"https://github.com/user-attachments/assets/b8b89b2f-3c4f-40c1-b5b9-88a1044208cf\"\n/>|
<video\nsrc=\"https://github.com/user-attachments/assets/33495309-c672-47c1-9f0d-e26892b62ef6\"\n/>|\n|
Borealis Timeline Minimized
|\n![image](https://github.com/user-attachments/assets/e07d0672-95ad-4e98-9695-c33064f5cc7d)|\n![image](https://github.com/user-attachments/assets/98dfc8d4-27e0-4e6a-82c0-117117c820d4)\n|\n|
Borealis Timeline Maximized ( remains unchanged )
|\n![image](https://github.com/user-attachments/assets/0e685d86-30d5-4fa0-8ed9-87ce98bac8a7)\n|\n![image](https://github.com/user-attachments/assets/978a9269-9180-4f92-96e9-257d0ae475e9)\n|","sha":"089a91c8287da8fc789e2144d778e7daddba44d1"}}]}]
BACKPORT-->

Co-authored-by: Jatin Kathuria <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport:prev-minor Backport to (8.x) the previous minor version (i.e. one version back from main) release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Investigations Team v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants