chartjs / chartjs-chart-financial

Chart.js module for charting financial securities
MIT License
721 stars 196 forks source link

[Bug report] Tooltip Bug for Mixed chart type candlestick & line #90

Open rashearth opened 3 years ago

rashearth commented 3 years ago

Hi, thanks for the migration for the chart.js 3.0 . I just found some bugs written in the title of this report.

please kindly check it through.

[what i done] I made a simple mixed chart of candlestick and line chart

[bugs]

Here, the tooltips have some issue when pointing the mouse to the line chart.(it seems to be OK in the candlestick)

there is many errors occuring, and the tooltip freezes+ wont show the right data. I assume that it has some compatibile issue with line chart {x: , y: ,} format and candlestick { t:,o:,h:,l:,c,} format

スクリーンショット 2020-12-12 13 52 59

[code] Here it is!

http://aifx.sakura.ne.jp/project/www/chartjs3.html

benmccann commented 3 years ago

I'm not seeing any errors in the developer console. I did make some changes recently and you seem to be linking directly to https://www.chartjs.org/chartjs-chart-financial/chartjs-chart-financial.js so perhaps it's been fixed

rashearth commented 3 years ago

Hi, thanks for the confirmation! As i have update the chart.js to betav9, it gives the below error, and the tooltip have some issues.

スクリーンショット 2021-01-22 23 19 29

please kindly confirm http://aifx.sakura.ne.jp/project/www/chartjs3.html

CupcakeThief commented 3 years ago

Yep I've hit the same issue. Yet another breaking change to v3.0.0, yay. Fine with beta.6, errors with beta.7

I stuck a breakpoint on the "if", but we don't seem to enter this function in beta.6 (and beta.7 we can see that Chart.Chart.defaults.interaction.callbacks is null:

if (!isNullOrUndef(point.y)) {
    return Chart.Chart.defaults.interaction.callbacks.label(ctx);
}
CreativeWarlock commented 3 years ago

Same here.