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

Fix point label counting in radialLinear scale #6280

Merged
merged 1 commit into from
May 16, 2019

Conversation

nagix
Copy link
Contributor

@nagix nagix commented May 16, 2019

If both angleLines.display and pointLabels.display are set to false, not only angle lines and point labels but also grid lines and data lines/points will be hidden.

Checks for angleLines.display and pointLabels.display in getValueCount are unnecessary because callers of this function are always checking these options if needed.

Master: https://jsfiddle.net/nagix/7etbk5jq/
Screen Shot 2019-05-16 at 3 19 49 PM

This PR: https://jsfiddle.net/nagix/g6x7Lt91/
Screen Shot 2019-05-16 at 3 19 23 PM

@simonbrunel simonbrunel added this to the Version 2.9 milestone May 16, 2019
@simonbrunel simonbrunel merged commit 9482ee5 into chartjs:master May 16, 2019
@nagix nagix deleted the issue-6280 branch May 16, 2019 18:50
exwm pushed a commit to exwm/Chart.js that referenced this pull request 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

Successfully merging this pull request may close these issues.

4 participants