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

[Security Solution] Reposition EuiPopovers on scroll #69433

Conversation

patrykkopycinski
Copy link
Contributor

@patrykkopycinski patrykkopycinski commented Jun 17, 2020

Summary

Fixes an issue found in 7.8.0 BC5 #655 and 7.7.1 #691

Initial Status:

  • To have at least one query saved in timelines

Steps to Reproduce:

  1. Open timeline toggle
  2. Click on the floppy disk icon
  3. Scroll down

Current Behaviour:

  • The Saved Queries dialog is scrolled along with the SIEM page.

Expected Behaviour:

  • The Saved Queries dialog is not scrolled along with the SIEM page.

Screenshot 2020-06-09 at 17 59 43

@patrykkopycinski patrykkopycinski added bug Fixes for quality problems that affect the customer experience Team:SIEM v8.0.0 v7.9.0 labels Jun 17, 2020
@patrykkopycinski patrykkopycinski self-assigned this Jun 17, 2020
@patrykkopycinski patrykkopycinski marked this pull request as ready for review June 22, 2020 08:01
@patrykkopycinski patrykkopycinski requested review from a team as code owners June 22, 2020 08:01
@elasticmachine
Copy link
Contributor

Pinging @elastic/siem (Team:SIEM)

@patrykkopycinski
Copy link
Contributor Author

@elasticmachine merge upstream

@spong
Copy link
Member

spong commented Jun 25, 2020

This appears to be related: #45021

@patrykkopycinski
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@streamich streamich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AppArch changes LGTM.

@patrykkopycinski
Copy link
Contributor Author

@elasticmachine merge upstream

Copy link
Contributor

@stephmilovic stephmilovic left a 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
@patrykkopycinski
Copy link
Contributor Author

@elasticmachine merge upstream

@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@patrykkopycinski patrykkopycinski merged commit 55922cb into elastic:master Jul 2, 2020
@patrykkopycinski patrykkopycinski deleted the fix/popover-reposition-on-scroll branch July 2, 2020 15:37
patrykkopycinski added a commit to patrykkopycinski/kibana that referenced this pull request Jul 2, 2020
* [Security Solution] Reposition EuiPopovers on scroll

* update snapshots

Co-authored-by: Elastic Machine <[email protected]>
patrykkopycinski added a commit to patrykkopycinski/kibana that referenced this pull request Jul 2, 2020
* [Security Solution] Reposition EuiPopovers on scroll

* update snapshots

Co-authored-by: Elastic Machine <[email protected]>
patrykkopycinski added a commit that referenced this pull request Jul 2, 2020
* [Security Solution] Reposition EuiPopovers on scroll

* update snapshots

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
patrykkopycinski added a commit that referenced this pull request Jul 2, 2020
* [Security Solution] Reposition EuiPopovers on scroll

* update snapshots

Co-authored-by: Elastic Machine <[email protected]>

Co-authored-by: Elastic Machine <[email protected]>
@MindyRS MindyRS added the Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. label Sep 23, 2021
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-solution (Team: SecuritySolution)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Fixes for quality problems that affect the customer experience release_note:fix Team: SecuritySolution Security Solutions Team working on SIEM, Endpoint, Timeline, Resolver, etc. Team:SIEM v7.9.0 v8.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants