chart.labels are not updating

[wchmiel]

Expected behaviour

chart.labels should update using chart.update() with new labels options.

Actual behaviour

chart.labels are not updating.

Live demo with steps to reproduce

https://jsfiddle.net/BlackLabel/fxuypzw5/

[sebastianbochan]

It looks like the renderLabels function does not set reference to rendered label. As a result update does not recognize object and does not apply new options.

Workaround
The solution is adding label by renderer.label, keep reference and call the attr() method as update.

Demo:

• https://jsfiddle.net/BlackLabel/vdo1sfkw/

Docs:

• https://api.highcharts.com/class-reference/Highcharts.SVGRenderer#text

[pawelfus]

Maybe we should slowly deprecate this chart.label in favour of annotations?

[sebastianbochan]

The second solution is keeping all labels in extra array like chart.labels or something. As a result reference to labels will exist.

Internal note
Regarding to pawel's idea, I agree but, how about back-compatibility?
@TorsteinHonsi what do you think?

[TorsteinHonsi]

I agree with Pawel, the chart.labels feature is so limited that we should rather slowly deprecate it in favour of annotations.

[sebastianbochan]

Will we focus on changing docs only or remove the functionality in core too?

I suggest to start with update docs at the beginning.

[TorsteinHonsi]

I suggest to start with update docs at the beginning.

Yes I agree. The functionality is so small that it doesn't hurt to leave it there.