Skip to content

[SecuritySolutuion] Fix misaligned toolbar in alerts table#196804

Merged
janmonschke merged 2 commits intoelastic:mainfrom
janmonschke:securitySolution/fix-misaligned-toolbar
Oct 21, 2024
Merged

[SecuritySolutuion] Fix misaligned toolbar in alerts table#196804
janmonschke merged 2 commits intoelastic:mainfrom
janmonschke:securitySolution/fix-misaligned-toolbar

Conversation

@janmonschke
Copy link
Contributor

@janmonschke janmonschke commented Oct 18, 2024

Summary

Some items in the alerts table's toolbar were misaligned. This PR adds a quick fix for these misaligned items but the actual fix will hopefully be done in EUI (elastic/eui#8085)

Before:
Screenshot 2024-10-18 at 09 47 53

After:
Screenshot 2024-10-18 at 09 47 14

Fixes #194521

@janmonschke janmonschke added backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes v9.0.0 Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v8.16.0 labels Oct 18, 2024
@janmonschke janmonschke marked this pull request as ready for review October 18, 2024 08:12
@janmonschke janmonschke requested a review from a team as a code owner October 18, 2024 08:12
@elasticmachine
Copy link
Contributor

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

@janmonschke
Copy link
Contributor Author

@elasticmachine merge upstream

@janmonschke janmonschke enabled auto-merge (squash) October 21, 2024 09:24
@jcger
Copy link
Contributor

jcger commented Oct 21, 2024

closes #194521

thanks Jan!

@janmonschke janmonschke merged commit 082a897 into elastic:main Oct 21, 2024
@kibanamachine
Copy link
Contributor

Starting backport for target branches: 8.16, 8.x

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

@elasticmachine
Copy link
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Test Failures

  • [job] [logs] FTR Configs #62 / Rule execution logic API Detection Engine - Execution logic @ess @serverless Indicator match type rules, alert suppression Code execution path: events count is greater than threats count should suppress an alert on real rule executions

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
triggersActionsUi 1.7MB 1.7MB +85.0B

History

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 21, 2024
…96804)

## Summary

Some items in the alerts table's toolbar were misaligned. This PR adds a
quick fix for these misaligned items but the actual fix will hopefully
be done in EUI (elastic/eui#8085)

Before:
<img width="1265" alt="Screenshot 2024-10-18 at 09 47 53"
src="https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf">

After:
<img width="1274" alt="Screenshot 2024-10-18 at 09 47 14"
src="https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a">

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 082a897)
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Oct 21, 2024
…96804)

## Summary

Some items in the alerts table's toolbar were misaligned. This PR adds a
quick fix for these misaligned items but the actual fix will hopefully
be done in EUI (elastic/eui#8085)

Before:
<img width="1265" alt="Screenshot 2024-10-18 at 09 47 53"
src="https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf">

After:
<img width="1274" alt="Screenshot 2024-10-18 at 09 47 14"
src="https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a">

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
(cherry picked from commit 082a897)
@kibanamachine
Copy link
Contributor

💚 All backports created successfully

Status Branch Result
8.16
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 Oct 21, 2024
…6804) (#197040)

# Backport

This will backport the following commits from `main` to `8.x`:
- [[SecuritySolutuion] Fix misaligned toolbar in alerts table
(#196804)](#196804)

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

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

<!--BACKPORT [{"author":{"name":"Jan
Monschke","email":"jan.monschke@elastic.co"},"sourceCommit":{"committedDate":"2024-10-21T11:49:39Z","message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","v8.16.0"],"title":"[SecuritySolutuion] Fix
misaligned toolbar in alerts
table","number":196804,"url":"https://github.com/elastic/kibana/pull/196804","mergeCommit":{"message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196804","number":196804,"mergeCommit":{"message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jan Monschke <jan.monschke@elastic.co>
kibanamachine added a commit that referenced this pull request Oct 21, 2024
…96804) (#197039)

# Backport

This will backport the following commits from `main` to `8.16`:
- [[SecuritySolutuion] Fix misaligned toolbar in alerts table
(#196804)](#196804)

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

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

<!--BACKPORT [{"author":{"name":"Jan
Monschke","email":"jan.monschke@elastic.co"},"sourceCommit":{"committedDate":"2024-10-21T11:49:39Z","message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218","branchLabelMapping":{"^v9.0.0$":"main","^v8.17.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["backport","release_note:skip","v9.0.0","Team:Threat
Hunting:Investigations","v8.16.0"],"title":"[SecuritySolutuion] Fix
misaligned toolbar in alerts
table","number":196804,"url":"https://github.com/elastic/kibana/pull/196804","mergeCommit":{"message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218"}},"sourceBranch":"main","suggestedTargetBranches":["8.16"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","branchLabelMappingKey":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/196804","number":196804,"mergeCommit":{"message":"[SecuritySolutuion]
Fix misaligned toolbar in alerts table (#196804)\n\n##
Summary\r\n\r\nSome items in the alerts table's toolbar were misaligned.
This PR adds a\r\nquick fix for these misaligned items but the actual
fix will hopefully\r\nbe done in EUI
(https://github.com/elastic/eui/pull/8085)\r\n\r\nBefore:\r\n<img
width=\"1265\" alt=\"Screenshot 2024-10-18 at 09 47
53\"\r\nsrc=\"https://github.com/user-attachments/assets/b5220e8f-f26e-4847-8326-504863bc39bf\">\r\n\r\nAfter:\r\n<img
width=\"1274\" alt=\"Screenshot 2024-10-18 at 09 47
14\"\r\nsrc=\"https://github.com/user-attachments/assets/ff6cbc31-301a-4b43-afc2-938262118d8a\">\r\n\r\nCo-authored-by:
Elastic Machine
<elasticmachine@users.noreply.github.com>","sha":"082a89797edaa45a5d20b5378f8d521561800218"}},{"branch":"8.16","label":"v8.16.0","branchLabelMappingKey":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->

Co-authored-by: Jan Monschke <jan.monschke@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting:Investigations Security Solution Threat Hunting Investigations Team v8.16.0 v8.17.0 v9.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Alert table] Number of alerts and last update position is a bit higher than the rest of the elements

4 participants