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][Notes] - disable note buttons in the right panel header when in preview mode (#199189) #199232

Merged
merged 1 commit into from
Nov 6, 2024

Conversation

PhilippeOberti
Copy link
Contributor

Backport

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

Questions ?

Please refer to the Backport tool documentation

…header when in preview mode (elastic#199189)

## Summary

This PR fixes a issue where the `Add note` button and the `+` icon
button are clickable when an alert is viewed in preview mode. Users
should not be able to perform actions here, as the action expands the
flyouts and opens the left panel Notes tab, but the issue is the left
panel now shows a different alert from the right panel. If the user
closes the preview panel, they now see a different alerts on the left
and right panels but they have no way to know this.

#### Add note button disabled

https://github.com/user-attachments/assets/20554b60-39a1-4c6d-b215-e502b5b24dbd

#### + button disabled

https://github.com/user-attachments/assets/df540aed-b583-457d-a9f4-0093a171ddaa

Also adding notes should be disabled when in the rule creation page, as
we do not want to generate notes for alerts that actually do not exist
yet. To be consistent with the other blocks in the flyout header, we
show a `-`.

https://github.com/user-attachments/assets/b62ecf85-ee0f-4bee-853c-ff1034b5bf25
(cherry picked from commit 53acbab)
@PhilippeOberti PhilippeOberti merged commit 31d7738 into elastic:8.x Nov 6, 2024
36 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 +240.0B

@PhilippeOberti PhilippeOberti deleted the backport/8.x/pr-199189 branch November 7, 2024 16:21
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