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

Pie chart hover doesn't work properly... #2983

Closed
FlashThunder opened this issue Jul 16, 2016 · 8 comments
Closed

Pie chart hover doesn't work properly... #2983

FlashThunder opened this issue Jul 16, 2016 · 8 comments

Comments

@FlashThunder
Copy link

I don't need to send any code, as in documentation it doesn't work:

http://www.chartjs.org/docs/#doughnut-pie-chart

Hover works only on top of chart, like 20px of it. But not at whole chart. So basically blue element is not hoverable.

@etimberg
Copy link
Member

Seems related #2389 For some reason the h3 covers the canvas. @nnnick any thoughts?

@theawesomenayak
Copy link
Contributor

theawesomenayak commented Jul 17, 2016

@etimberg How can I test the changes made to docs?

@etimberg
Copy link
Member

@nnnick is the only one who can build the docs unfortunately.

@FlashThunder
Copy link
Author

FlashThunder commented Jul 17, 2016

and how do I get code from documentation examples??? this is kinda annoying... how do I make legend like that? how do I setup pie chart that would look and behave exactly like in documentation? I got only chunks of code there and no full case...

samples on GitHub don't even work... they are pointing to non-existing javascripts (like ../dist/Chart.bundle.js)

@etimberg
Copy link
Member

@FlashThunder do a build (using gulp build) and the files in the dist folder will be there. We've stopped checking them into git.

@theawesomenayak
Copy link
Contributor

@etimberg So can I submit a PR with the fix? I have tested the changes by editing the HTML on the docs page and know for sure that it works.

@etimberg
Copy link
Member

@sibennayak sure, submit a PR. I just don't know where since only @nnnick has the templates

etimberg added a commit that referenced this issue Jul 18, 2016
Fix the Pie Chart hover issue (#2983)
@etimberg
Copy link
Member

Fixed in #2986

exwm pushed a commit to exwm/Chart.js that referenced this issue Apr 30, 2021
exwm pushed a commit to exwm/Chart.js that referenced this issue Apr 30, 2021
exwm pushed a commit to exwm/Chart.js that referenced this issue Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants