blacklabel / indicators

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

2 Yaxis on same chart => split on redraw #63

Closed Kurairaito closed 7 years ago

Kurairaito commented 7 years ago

Hi,

Here is an example : https://codepen.io/anon/pen/zEqaZx don't pay attention to x axis, I took an existing example to show you the bug.

When using more that 1 Yaxis in the same graph, we get 2 graph instead of 1 originally when redrawing. for the full example press the ATR or RSI button at the bottom.

Regards.

pawelfus commented 7 years ago

Hi @Kurairaito

Thank you for contacting us. This is caused by chart.alignAxes option:

true/false. When indicator requires separate yAxis (like ATR or RSI) then indicators plugin will update heights of the axes to fit plotting area. Disable it and set yAxis.top and yAxis.height when you want different heights for the y-axes. Defaults to true. Note: When disabled, top and height options need to be set for all axes for better view.

Indicators in this mode will always split all axes and fit them evenly in the container. In your case I suggest to disable this and manually manage all yAxis heights/tops.

Kurairaito commented 7 years ago

Hi,

thanks for your answer, I almost succeed in manually manage height and top but I have an issue in this new example : https://codepen.io/anon/pen/bopOMr the issue is the line 158 doesn't seems to update the position of the indicator axis, and I didn't found any function to do that.

To reproduce : active RSI, active ATR, then disable RSI => in the console, I print the modified Yaxis for the remaining indicator (atr), it's to is set to 75% but in the result, it stayed at 50%.

regards.

Kurairaito commented 7 years ago

well I found it by myself, it's the exact same issue as https://github.com/blacklabel/indicators/issues/61 and the last answer it the right solution.