Skip to content

[8.18] [Security Solution] Fix session view navigation when in alert preview and add preview banner (#213455)#214467

Merged
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-213455
Mar 13, 2025
Merged

[8.18] [Security Solution] Fix session view navigation when in alert preview and add preview banner (#213455)#214467
kibanamachine merged 1 commit intoelastic:8.18from
kibanamachine:backport/8.18/pr-213455

Conversation

@kibanamachine
Copy link
Contributor

Backport

This will backport the following commits from main to 8.18:

Questions ?

Please refer to the Backport tool documentation

… and add preview banner (elastic#213455)

## Summary

Before: when in an alert preview, clicking session preview header opens
session view in details panel, but the preview persists.

This PR addressed the bug by including `isPreviewMode` as an indicator
that the panel is opened in preview. Clicking a link in preview should
opens a new flyout and preview should not appear.

https://github.com/user-attachments/assets/484daa16-adb0-48f2-b14e-a971878817c0

Added banner to session viewer in flyout

<img width="1468" alt="image"
src="https://github.com/user-attachments/assets/e228b35f-938a-4ad1-a97f-818caf36a284"
/>

### Checklist

- [x] Any text added follows [EUI's writing
guidelines](https://elastic.github.io/eui/#/guidelines/writing), uses
sentence case text and includes [i18n
support](https://github.com/elastic/kibana/blob/main/src/platform/packages/shared/kbn-i18n/README.md)
- [ ] [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
- [ ] The PR description includes the appropriate Release Notes section,
and the correct `release_note:*` label is applied per the
[guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process)

(cherry picked from commit a6ceafc)
@kibanamachine kibanamachine added the backport This PR is a backport of another PR label Mar 13, 2025
@kibanamachine kibanamachine enabled auto-merge (squash) March 13, 2025 18:49
@kibanamachine kibanamachine merged commit 319e8f2 into elastic:8.18 Mar 13, 2025
11 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 9.1MB 9.1MB +243.0B

cc @christineweng

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants