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

[SIEM] Makes ip details heading draggable #45179

Merged

Conversation

stephmilovic
Copy link
Contributor

Summary

Resolves #40621

When in the IP Details view, the only place the IP address was shown is in the header, but it was not draggable to the Timeline. This PR enables dragging on headers when passed draggable values, and implements the component on IP Details.
f

Checklist

Use strikethroughs to remove checklist items you don't feel are applicable to this PR.

For maintainers

@stephmilovic stephmilovic added Team:SIEM v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.4.0 labels Sep 9, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/siem

@stephmilovic stephmilovic changed the title ip details heading draggable [SIEM] ip details heading draggable Sep 9, 2019
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@andrew-goldstein andrew-goldstein left a comment

Choose a reason for hiding this comment

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

Approved with an optional, but recommended change from DraggableWrapper to DefaultDraggable.

LGTM 🙏

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@stephmilovic stephmilovic merged commit 585b548 into elastic:master Sep 10, 2019
@stephmilovic stephmilovic deleted the ip-details-heading-draggable branch September 10, 2019 03:27
stephmilovic added a commit to stephmilovic/kibana that referenced this pull request Sep 10, 2019
stephmilovic added a commit to stephmilovic/kibana that referenced this pull request Sep 10, 2019
jloleysens added a commit to jloleysens/kibana that referenced this pull request Sep 10, 2019
…-to-np-ready

* 'master' of github.com:elastic/kibana: (138 commits)
  [Canvas] i18n work on workpad header (and a few header CTAs) and convert to typescript (elastic#44943)
  update close/delete system index modals (elastic#45037)
  TS return type of createIndexPatternSelect (elastic#45107)
  [ML] Fix focus chart updating. (elastic#45146)
  [ML] Data frame transform: Fix progress in wizard create step. (elastic#45116)
  [Graph] Re-enable functional test (elastic#44683)
  [SIEM] unique table id for each top talkers table (elastic#45014)
  [SIEM] ip details heading draggable (elastic#45179)
  [Maps][File upload] Set complete on index pattern creation (elastic#44423)
  [Maps] unmount map embeddable component on destroy (elastic#45183)
  [SIEM] Adds error toasts to MapEmbeddable component (elastic#45088)
  fix redirect to maintain search query string (elastic#45184)
  [APM] One-line trace summary (elastic#44842)
  [Infra UI] Display non-metric details on Node Detail page (elastic#43551)
  [Maps][File upload] Removing bbox from parsed file pending upstream lib fix (elastic#45194)
  [Logs UI] Improve live streaming behavior when scrolling (elastic#44923)
  [APM] Fix indefinite loading state in agent settings for unauthorized user roles (elastic#44970)
  [Reporting] Rewrite addForceNowQuerystring to getFullUrls (elastic#44851)
  [Reporting/ESQueue] Improve logging of doc-update events (elastic#45077)
  [Reporting] Make screenshot capture less noisy by default (elastic#45185)
  ...
@FrankHassanabad FrankHassanabad added v7.5.0 release_note:enhancement and removed release_note:skip Skip the PR/issue when compiling release notes labels Sep 16, 2019
@FrankHassanabad FrankHassanabad changed the title [SIEM] ip details heading draggable [SIEM] Makes ip details heading draggable Sep 16, 2019
@elasticmachine
Copy link
Contributor

💔 Build Failed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[SIEM] Timeline: Drag IP from IP Detail view
4 participants