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] Bring back Drag drop in query tabs of active timeline #111908

Merged
merged 8 commits into from
Sep 13, 2021

Conversation

XavierM
Copy link
Contributor

@XavierM XavierM commented Sep 10, 2021

Summary

We realized during testing that we were not able to drag and drop value from the host/network/event details inside of the active timeline. As a team we decided that user should still have this functionality in the query tab of the active timeline.

Checklist

@XavierM XavierM added enhancement New value added to drive a business result release_note:skip Skip the PR/issue when compiling release notes v7.8.0 Team:Threat Hunting Security Solution Threat Hunting Team auto-backport Deprecated - use backport:version if exact versions are needed v7.16.0 v7.15.1 labels Sep 10, 2021
@XavierM XavierM requested a review from a team as a code owner September 10, 2021 21:39
@elasticmachine
Copy link
Contributor

Pinging @elastic/security-threat-hunting (Team:Threat Hunting)

@kqualters-elastic
Copy link
Contributor

desk tested locally. drag and drop works as before for the event detail panel when on the query tab, and is disabled on all the others. tried to break it by switching to different tabs while the detail panel was open but drag and drop was enabled and disabled as appropriate. failing snapshot seems like it will pass once it's updated, lgtm

@XavierM XavierM added v7.15.0 and removed v7.15.1 labels Sep 13, 2021
@XavierM XavierM enabled auto-merge (squash) September 13, 2021 17:41
@kibanamachine
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 6.5MB 6.5MB +1.3KB

History

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

@XavierM XavierM merged commit 35c83ba into elastic:master Sep 13, 2021
@kibanamachine
Copy link
Contributor

The following labels were identified as gaps in your version labels and will be added automatically:

  • v7.9.0
  • v7.10.0
  • v7.11.0
  • v7.12.0
  • v7.13.0
  • v7.14.0

If any of these should not be on your pull request, please manually remove them.

kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 13, 2021
…line (elastic#111908)

* bring back drag drop in active timeline detail view

* only allow drag and drop in query tab

* fix snapshot

* fix bug on alert summary to remove dragdrop
kibanamachine pushed a commit to kibanamachine/kibana that referenced this pull request Sep 13, 2021
…line (elastic#111908)

* bring back drag drop in active timeline detail view

* only allow drag and drop in query tab

* fix snapshot

* fix bug on alert summary to remove dragdrop
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
7.10 Commit could not be cherrypicked due to conflicts
7.8 Commit could not be cherrypicked due to conflicts
7.9 Commit could not be cherrypicked due to conflicts
7.11 Commit could not be cherrypicked due to conflicts
7.12 Commit could not be cherrypicked due to conflicts
7.14 Commit could not be cherrypicked due to conflicts
7.13 Commit could not be cherrypicked due to conflicts
7.15
7.x

Successful backport PRs will be merged automatically after passing CI.

To backport manually run:
node scripts/backport --pr 111908

kibanamachine added a commit that referenced this pull request Sep 13, 2021
…line (#111908) (#112023)

* bring back drag drop in active timeline detail view

* only allow drag and drop in query tab

* fix snapshot

* fix bug on alert summary to remove dragdrop

Co-authored-by: Xavier Mouligneau <[email protected]>
kibanamachine added a commit that referenced this pull request Sep 14, 2021
…line (#111908) (#112024)

* bring back drag drop in active timeline detail view

* only allow drag and drop in query tab

* fix snapshot

* fix bug on alert summary to remove dragdrop

Co-authored-by: Xavier Mouligneau <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed enhancement New value added to drive a business result release_note:skip Skip the PR/issue when compiling release notes Team:Threat Hunting Security Solution Threat Hunting Team v7.8.0 v7.9.0 v7.10.0 v7.11.0 v7.12.0 v7.13.0 v7.14.0 v7.15.0 v7.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants