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

[8.x] [Security Solution][Alert details] - refactor UI on insights (#197349) #198417

Merged
merged 1 commit into from
Oct 30, 2024

Conversation

kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.x:

Questions ?

Please refer to the Backport tool documentation

…#197349)

## Summary

This PR refactors the Insights section of the expandable flyout for
alerts and events.

The changes are applied to the following section:
- Threat Intelligence: when the user clicks on the number, we open the
left section to the Insights Threat Intelligence tab
- Correlations: when the user clicks on the number, we open the left
section to the Insights Correlations tab
- Prevalence: no user interactions

When in preview mode, none of the number are clickable and the buttons
are disabled.

#### New UI

| Normal flyout  | Preview flyout |
| ------------- | ------------- |
| ![Screenshot 2024-10-22 at 6 01
38 PM](https://github.com/user-attachments/assets/de179a2b-c8ab-42f6-b5b7-839dae0139d5)
| ![Screenshot 2024-10-22 at 6 01
54 PM](https://github.com/user-attachments/assets/63ed125e-5e3b-4c4c-a10e-7cc01d291660)
|

#### UX flows to expand the flyout

https://github.com/user-attachments/assets/30031a12-c2f3-47e6-a783-5b9482359ee5

elastic/security-team#7033

### Checklist

- [x] [Unit or functional
tests](https://www.elastic.co/guide/en/kibana/master/development-tests.html)
were updated or added to match the most common scenarios

(cherry picked from commit 9b9eb7e)
@kibanamachine kibanamachine merged commit 85090ca into elastic:8.x Oct 30, 2024
38 checks passed
@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.0MB 21.0MB +1.7KB

cc @PhilippeOberti

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants