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

[Logs UI] Change tool tip labels for expected boundaries #45297

Closed
simianhacker opened this issue Sep 10, 2019 · 5 comments
Closed

[Logs UI] Change tool tip labels for expected boundaries #45297

simianhacker opened this issue Sep 10, 2019 · 5 comments
Labels
blocked enhancement New value added to drive a business result Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services

Comments

@simianhacker
Copy link
Member

simianhacker commented Sep 10, 2019

Note: This work is blocked by elastic/elastic-charts#162

Since we can't hide data points in the tool tip we should change the labels to indicate what part of the boundaries they refer to. Instead of just listing both the boundaries as Expected we should change the respective labels to be Upper Bounds and Lower Bounds. This will be consistent to how the ML team refers to those values in their charts.

Logs UI Tool tip

image

ML Tool tip

image

@simianhacker simianhacker added enhancement New value added to drive a business result Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services labels Sep 10, 2019
@elasticmachine
Copy link
Contributor

Pinging @elastic/infra-logs-ui

@jasonrhodes
Copy link
Member

I updated the description, but FYI this work is blocked by elastic/elastic-charts#162

@markov00
Copy link
Member

Hey @jasonrhodes what will be the optimal solution:

  1. hide the upper and lower bound values on the tooltip
  2. change the current behaviour and add the possibility to customize the tooltip names to upper/lower bounds?

@jasonrhodes
Copy link
Member

@markov00 I'd say (2) is the more commonly needed solution (you probably want to tell people those bound values in most cases) but I can imagine wanting option (1) at times as well. Although that's more of a "how can I hide a series from the tooltip" question I think, which is applicable across all series imo.

@Kerry350
Copy link
Contributor

Closing this specific Logs UI ticket as we're no longer rendering a band area chart for this data visualisation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked enhancement New value added to drive a business result Feature:Logs UI Logs UI feature Team:Infra Monitoring UI - DEPRECATED DEPRECATED - Label for the Infra Monitoring UI team. Use Team:obs-ux-infra_services
Projects
None yet
Development

No branches or pull requests

7 participants