Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
rashearth opened this issue Dec 12, 2020 · 4 comments
Open

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

rashearth opened this issue Dec 12, 2020 · 4 comments

Comments

@rashearth
Copy link

rashearth commented Dec 12, 2020

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
Copy link
Collaborator

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
Copy link
Author

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
Copy link
Contributor

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
Copy link

Same here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants