-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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] Reposition EuiPopovers on scroll #69433
[Security Solution] Reposition EuiPopovers on scroll #69433
Conversation
…eposition-on-scroll
…eposition-on-scroll
Pinging @elastic/siem (Team:SIEM) |
@elasticmachine merge upstream |
This appears to be related: #45021 |
@elasticmachine merge upstream |
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.
AppArch changes LGTM.
@elasticmachine merge upstream |
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.
manual testing and code review both LGTM, thanks @patrykkopycinski ! 🥳
…eposition-on-scroll # Conflicts: # x-pack/plugins/security_solution/public/endpoint_alerts/view/details/overview/take_action_dropdown.tsx
@elasticmachine merge upstream |
💚 Build SucceededBuild metrics
History
To update your PR or re-run it, just comment with: |
* [Security Solution] Reposition EuiPopovers on scroll * update snapshots Co-authored-by: Elastic Machine <[email protected]>
* [Security Solution] Reposition EuiPopovers on scroll * update snapshots Co-authored-by: Elastic Machine <[email protected]>
* [Security Solution] Reposition EuiPopovers on scroll * update snapshots Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
* [Security Solution] Reposition EuiPopovers on scroll * update snapshots Co-authored-by: Elastic Machine <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Pinging @elastic/security-solution (Team: SecuritySolution) |
Summary
Fixes an issue found in 7.8.0 BC5 #655 and 7.7.1 #691
Initial Status:
Steps to Reproduce:
floppy disk
iconCurrent Behaviour:
Saved Queries
dialog is scrolled along with the SIEM page.Expected Behaviour:
Saved Queries
dialog is not scrolled along with the SIEM page.