Group by text displayed in some charts is truncated

[madalincm]

Steps to Reproduce:

1. Open https://pipeline-sql.stage.mozaws.net/queries/374#460
2. Hover over any of the bars in chart

Expected results:
All the information are displayed completely. For example when hovering over the blue bar the text: "App 2.0.1 - Android 22" is correctly displayed

Actual results:
When hovering over the blue bar the ""App 2.0.1 - Android 22"" text is truncated and only "App 2.0.1 - ..." is displaued

Notes/Issues:
Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:59.0) Gecko/20100101 Firefox/59.0
Verified in FF59(Win7). Issue can be reproduced in STMO-stage and STMO-prod
Production query where this issue can be reproduced: https://sql.telemetry.mozilla.org/queries/52806#140500
Screencast for this issue:

[alison985]

I can reproduce this issue on redash v4.0.1+b838f28f if there are multiple series. On single series bar charts it doesn't even try to display the label.

In order to see the HTML generated on hover you have to use Chrome and the "Break on > subtree modifications" feature. That points you to the d3.js file.

Plotly.js supports hover text: https://plot.ly/javascript/bar-charts/#bar-chart-with-hover-text
I forked the codepen example and changed the plotly.js down to our version - 1.20.1 here: https://codepen.io/plotly/pen/c11f764ad9de0a40f3f4989517195cf0 (it's not saving the version change) It does allow untruncated labels to be passed into the text parameter.

Adding the text parameter and the name of the series to the plotlySeries in javascript (utils.js) does not fix the issue. hoverinfo and hovertext also do not fix the issue.

Searching for '...', which is what is being appended to each label after truncation does not result in any leads.

[madalincm]

Verified in FF60(Win7) in stmo-stage
The original issue appears to be fixed, the entire string is displayed:

But as can be observed in the above screenshot there is a problem with the text alignment.
I will mark this issue as verified fixed an open an new one to track that.

[jezdez]

Reopening since we decided to roll back #422 following a discussion with @emtwo on IRC about #437.

The real culprit seems to be a missing feature in plotly (plotly/plotly.js#460) and the best I was able to get to was:

I'm really wondering if it's not enough to have the full label in the legend next to the graph?

[madalincm]

@jezdez Having the value is what really counts so I agree with you that having the label correctly displayed next to the graph should be ok for now.

[alison985]

@jezdez Are we closing as won't fix?

[jezdez]

@alison985 Yeah, with @madalincm's reply above I think we can close this. At least until plotly/plotly.js#460 is fixed.