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

Show values option is not working on horizontal bar chart #59076

Closed
rayafratkina opened this issue Mar 2, 2020 · 5 comments · Fixed by #78154
Closed

Show values option is not working on horizontal bar chart #59076

rayafratkina opened this issue Mar 2, 2020 · 5 comments · Fixed by #78154
Labels
enhancement New value added to drive a business result Feature:ElasticCharts Issues related to the elastic-charts library Feature:XYAxis XY-Axis charts (bar, area, line) Team:Visualizations Visualization editors, elastic-charts and infrastructure

Comments

@rayafratkina
Copy link
Contributor

Kibana version:
7.6.0

Elasticsearch version:
7.6.0

Cloud instance

Describe the bug:
Values not displayed on the chart when setting is selected

Steps to reproduce:

  1. Create a horizontal bar chart
  2. In the Panel options tab select "Show values on chart"
  3. No values are displayed

Expected behavior:
Values are displayed next to the bars

Screenshots (if relevant):
screencapture-a4f26f2e0db06fc318967585073af056-us-east-1-aws-found-io-9243-s-2018-in-review-app-kibana-2020-03-02-15_01_41

@rayafratkina rayafratkina added bug Fixes for quality problems that affect the customer experience triage_needed Team:Visualizations Visualization editors, elastic-charts and infrastructure labels Mar 2, 2020
@elasticmachine
Copy link
Contributor

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

@rayafratkina rayafratkina added the Feature:Vis Editor Visualization editor issues label Mar 2, 2020
@timroes timroes added Feature:XYAxis XY-Axis charts (bar, area, line) and removed Feature:Vis Editor Visualization editor issues bug Fixes for quality problems that affect the customer experience labels Mar 3, 2020
@timroes
Copy link
Contributor

timroes commented Mar 3, 2020

We only draw numbers beside the bars, if the number fits given it's size within the height (or width, if vertical) of the bar. I've just validated 7.6, and if you reduce the number of bars, and thus the bars become wider, you should start seeing labels again. This is by design, since we otherwise have potentially overlapping numbers beside the bars.

I'll still leave this open for some discussion on how this situation could be improved also for small bars (maybe only showing alternating numbers)?

cc @markov00 I think however we want to improve this, we don't want to make those changes in vislib anymore, but wait for elastic-charts to implement them.

@timroes timroes added enhancement New value added to drive a business result Feature:ElasticCharts Issues related to the elastic-charts library and removed triage_needed labels Mar 3, 2020
@rayafratkina
Copy link
Contributor Author

I understand that when displaying a chart, but when I am editing one I think at the very least we need a way to indicate that limitation to the end user.

+1 on doing it in the elastic charts, of course

@flash1293
Copy link
Contributor

As a reference, this is how it looks like in heatmap:
Screenshot 2020-03-03 at 16 14 05

@markov00
Copy link
Member

markov00 commented Mar 4, 2020

We have few options in elastic-charts for that:

  • hide clipped text (text that goes outside the chart area)
  • you can apply an option to alternate the text on bars
  • you can have the option to show only text inside the bar element or also outside
  • we have some (tricky and improvable) way to add offset to X and Y position of the labels

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New value added to drive a business result Feature:ElasticCharts Issues related to the elastic-charts library Feature:XYAxis XY-Axis charts (bar, area, line) Team:Visualizations Visualization editors, elastic-charts and infrastructure
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants