blacklabel / indicators

Indicators plugin for Highstock by Black Label
http://blacklabel.github.io/indicators/
Other
35 stars 12 forks source link

After destroying an indicator, tooltip of the connected series still has the indicator info. #51

Closed KacperMadej closed 7 years ago

KacperMadej commented 7 years ago

As in the title.

Demo: http://jsfiddle.net/RE7sS/395/

Steps:

  1. Click the button to remove ATR
  2. Hover over series to find ATR in tooltip

Workaround (by pawelfus) for dynamically added indicators:

Highcharts.charts[0].series[0].isDirtyData = true;
Highcharts.charts[0].indicators.allItems[0].destroy(false);
Highcharts.charts[0].redraw();

Demo: http://jsfiddle.net/nz0peu33/8/

KacperMadej commented 7 years ago

Fix suggestion: in Indicator.prototype.destroy() use tooltipkey.

Demo: http://jsfiddle.net/uc7qak5o/1/

lakshmikant4u commented 7 years ago

@pawelfus When can we expect new release which has this fix? It would be great if we have it ASAP.

pawelfus commented 7 years ago

Hi @lakshmikant4u

Thank you for the reminder. Version 2.1.5 released.

lakshmikant4u commented 7 years ago

Thanks @pawelfus .