-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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] Fix - Notes Flyout Product Feedback #188129
Conversation
Pinging @elastic/security-threat-hunting-investigations (Team:Threat Hunting:Investigations) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
code LGTM, thanks for updating this and fixing a bug at the same time!
⏳ Build in-progress, with failures
Failed CI StepsTest Failures
History
|
💔 All backports failed
Manual backportTo create the backport manually run:
Questions ?Please refer to the Backport tool documentation |
# Summary Fixes below bugs based on feedback from @paulewing. ## Event Details Toggle in Notes @paulewing requested to remove the event toggle |Before|After| |---|---| |![Bildschirmfoto 2024-07-11 um 17 48 15](https://github.com/elastic/kibana/assets/7485038/2b45d3a9-6f1a-4f05-8824-10e2c6265266)| ![Bildschirmfoto 2024-07-11 um 17 46 01](https://github.com/elastic/kibana/assets/7485038/b02c06ff-f556-4894-a588-a88bcdd8bc8c)| ## Notes Flyout remains open when switching tabs |Before|After| |---|---| |<video src="https://github.com/elastic/kibana/assets/7485038/4228d2d6-c2ad-40dc-9e6c-ec049f834e8f" />|<video src="https://github.com/elastic/kibana/assets/7485038/0e010c22-4539-4428-9b1b-3b323a9f491c" />| ## Notes Flyout should be resizable As shown in above video, notes flyout is now resizable. (cherry picked from commit 309b907) # Conflicts: # x-pack/plugins/security_solution/public/timelines/components/timeline/tabs/query/query_tab_unified_components.test.tsx
💚 All backports created successfully
Note: Successful backport PRs will be merged automatically after passing CI. Questions ?Please refer to the Backport tool documentation |
…) (#188401) # Backport This will backport the following commits from `main` to `8.15`: - [[Security Solution] Fix - Notes Flyout Product Feedback (#188129)](#188129) <!--- Backport version: 8.9.8 --> ### Questions ? Please refer to the [Backport tool documentation](https://github.com/sqren/backport) <!--BACKPORT [{"author":{"name":"Jatin Kathuria","email":"jatin.kathuria@elastic.co"},"sourceCommit":{"committedDate":"2024-07-12T17:20:19Z","message":"[Security Solution] Fix - Notes Flyout Product Feedback (#188129)\n\n# Summary\r\n\r\nFixes below bugs based on feedback from @paulewing.\r\n\r\n\r\n## Event Details Toggle in Notes\r\n\r\n@paulewing requested to remove the event toggle \r\n\r\n|Before|After|\r\n|---|---|\r\n|![Bildschirmfoto 2024-07-11 um 17 48\r\n15](https://github.com/elastic/kibana/assets/7485038/2b45d3a9-6f1a-4f05-8824-10e2c6265266)|\r\n![Bildschirmfoto 2024-07-11 um 17 46\r\n01](https://github.com/elastic/kibana/assets/7485038/b02c06ff-f556-4894-a588-a88bcdd8bc8c)|\r\n\r\n\r\n## Notes Flyout remains open when switching tabs\r\n|Before|After|\r\n|---|---|\r\n|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/4228d2d6-c2ad-40dc-9e6c-ec049f834e8f\"\r\n/>|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/0e010c22-4539-4428-9b1b-3b323a9f491c\"\r\n/>|\r\n\r\n\r\n## Notes Flyout should be resizable\r\n\r\nAs shown in above video, notes flyout is now resizable.","sha":"309b907e59df245236c24f7a3b121488da9dc3e4","branchLabelMapping":{"^v8.16.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","Feature:Timeline","Team:Threat Hunting:Investigations","backport:prev-minor","v8.16.0"],"number":188129,"url":"https://github.com/elastic/kibana/pull/188129","mergeCommit":{"message":"[Security Solution] Fix - Notes Flyout Product Feedback (#188129)\n\n# Summary\r\n\r\nFixes below bugs based on feedback from @paulewing.\r\n\r\n\r\n## Event Details Toggle in Notes\r\n\r\n@paulewing requested to remove the event toggle \r\n\r\n|Before|After|\r\n|---|---|\r\n|![Bildschirmfoto 2024-07-11 um 17 48\r\n15](https://github.com/elastic/kibana/assets/7485038/2b45d3a9-6f1a-4f05-8824-10e2c6265266)|\r\n![Bildschirmfoto 2024-07-11 um 17 46\r\n01](https://github.com/elastic/kibana/assets/7485038/b02c06ff-f556-4894-a588-a88bcdd8bc8c)|\r\n\r\n\r\n## Notes Flyout remains open when switching tabs\r\n|Before|After|\r\n|---|---|\r\n|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/4228d2d6-c2ad-40dc-9e6c-ec049f834e8f\"\r\n/>|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/0e010c22-4539-4428-9b1b-3b323a9f491c\"\r\n/>|\r\n\r\n\r\n## Notes Flyout should be resizable\r\n\r\nAs shown in above video, notes flyout is now resizable.","sha":"309b907e59df245236c24f7a3b121488da9dc3e4"}},"sourceBranch":"main","suggestedTargetBranches":[],"targetPullRequestStates":[{"branch":"main","label":"v8.16.0","labelRegex":"^v8.16.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/188129","number":188129,"mergeCommit":{"message":"[Security Solution] Fix - Notes Flyout Product Feedback (#188129)\n\n# Summary\r\n\r\nFixes below bugs based on feedback from @paulewing.\r\n\r\n\r\n## Event Details Toggle in Notes\r\n\r\n@paulewing requested to remove the event toggle \r\n\r\n|Before|After|\r\n|---|---|\r\n|![Bildschirmfoto 2024-07-11 um 17 48\r\n15](https://github.com/elastic/kibana/assets/7485038/2b45d3a9-6f1a-4f05-8824-10e2c6265266)|\r\n![Bildschirmfoto 2024-07-11 um 17 46\r\n01](https://github.com/elastic/kibana/assets/7485038/b02c06ff-f556-4894-a588-a88bcdd8bc8c)|\r\n\r\n\r\n## Notes Flyout remains open when switching tabs\r\n|Before|After|\r\n|---|---|\r\n|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/4228d2d6-c2ad-40dc-9e6c-ec049f834e8f\"\r\n/>|<video\r\nsrc=\"https://github.com/elastic/kibana/assets/7485038/0e010c22-4539-4428-9b1b-3b323a9f491c\"\r\n/>|\r\n\r\n\r\n## Notes Flyout should be resizable\r\n\r\nAs shown in above video, notes flyout is now resizable.","sha":"309b907e59df245236c24f7a3b121488da9dc3e4"}}]}] BACKPORT-->
Summary
Fixes below bugs based on feedback from @paulewing.
Event Details Toggle in Notes
@paulewing requested to remove the event toggle
Notes Flyout remains open when switching tabs
Bildschirmaufnahme.2024-07-11.um.17.54.18.mov
Bildschirmaufnahme.2024-07-11.um.17.56.19.mov
Notes Flyout should be resizable
As shown in above video, notes flyout is now resizable.