Skip to content

[Timelion] Vertical cursor is not displayed across visualizations of a dashboard#83435

Merged
VladLasitsa merged 1 commit intoelastic:masterfrom
VladLasitsa:#82250
Nov 18, 2020
Merged

[Timelion] Vertical cursor is not displayed across visualizations of a dashboard#83435
VladLasitsa merged 1 commit intoelastic:masterfrom
VladLasitsa:#82250

Conversation

@VladLasitsa
Copy link
Copy Markdown
Contributor

Closes: #82250

Summary

Added eventBus to trigger and listen plotHandler event so that we can set crosshair line in several instances of timelion.

Checklist

Delete any items that are not applicable to this PR.

For maintainers

@ghost
Copy link
Copy Markdown

ghost commented Nov 16, 2020

💚 Build Succeeded

@kibanamachine
Copy link
Copy Markdown
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
visTypeTimelion 74.4KB 74.8KB +493.0B

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

@stratoula stratoula changed the title Vertical cursor is not displayed across Timelion visualizations of a dashboad Vertical cursor is not displayed across Timelion visualizations of a dashboard Nov 17, 2020
@stratoula stratoula changed the title Vertical cursor is not displayed across Timelion visualizations of a dashboard [Timelion] Vertical cursor is not displayed across visualizations of a dashboard Nov 17, 2020
Copy link
Copy Markdown
Contributor

@stratoula stratoula left a comment

Choose a reason for hiding this comment

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

Code LGTM, I have tested locally and it works fine. Thanx @VladLasitsa for this PR. Can you update the description of this PR and also I would like to backport it on 7.10.1 too

@VladLasitsa VladLasitsa added v7.10.1 v7.11.0 v8.0.0 release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// Feature:Timelion Timelion app and visualization and removed v7.11.0 labels Nov 17, 2020
@VladLasitsa VladLasitsa marked this pull request as ready for review November 17, 2020 13:02
@VladLasitsa VladLasitsa requested a review from a team November 17, 2020 13:02
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/kibana-app (Team:KibanaApp)

@VladLasitsa VladLasitsa merged commit 6ff61c0 into elastic:master Nov 18, 2020
VladLasitsa added a commit to VladLasitsa/kibana that referenced this pull request Nov 18, 2020
VladLasitsa added a commit to VladLasitsa/kibana that referenced this pull request Nov 18, 2020
phillipb added a commit to phillipb/kibana that referenced this pull request Nov 18, 2020
…o-node-details

* 'master' of github.com:elastic/kibana: (65 commits)
  update chromedriver dependency to 87 (elastic#83624)
  [TSVB] use new Search API for rollup search (elastic#83275)
  [TSVB] Y-axis has number formatting not considering all series formatters in the group (elastic#83438)
  [Logs UI] Update <LogStream /> internal state when its props change (elastic#83302)
  Add tag bulk action context menu (elastic#82816)
  [code coverage] adding plugin to flush coverage data (elastic#83447)
  [UsageCollection] Expose `KibanaRequest` to explicitly opted-in collectors (elastic#83413)
  Added eventBus to trigger and listen plotHandler event (elastic#83435)
  [Runtime fields] Editor phase 1 (elastic#81472)
  [Maps] Fix threshold alert issue resolving nested fields (elastic#83577)
  chore(NA): remove usage of unverified es snapshots (elastic#83589)
  [DOCS] Adds Elastic Contributor Program link (elastic#83561)
  Upgrade EUI to v30.2.0 (elastic#82730)
  Don't show loading screen during auto-reload (elastic#83376)
  Functional tests - fix esArchive mappings with runtime fields (elastic#83530)
  [deb/rpm] Create keystore after installation (elastic#76465)
  [rpm] Create default environment file at "/etc/sysconfig/kibana" (elastic#82144)
  [docker] removes workaround for missing crypto-policies-scripts subpackage (elastic#83455)
  [ML] Persisted URL state for the Data frame analytics jobs and models pages (elastic#83439)
  adds xpack.security.authc.selector.enabled setting (elastic#83551)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature:Timelion Timelion app and visualization release_note:fix Team:Visualizations Team label for Lens, elastic-charts, Graph, legacy editors (TSVB, Visualize, Timelion) t// v7.10.1 v7.11.0 v8.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Vertical cursor is not displayed across Timelion visualizations of a dashboad

5 participants