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] lineTension doesn't work for Radar chart type. #3591

Closed
KoyoSE opened this issue Nov 13, 2016 · 1 comment
Closed

[BUG] lineTension doesn't work for Radar chart type. #3591

KoyoSE opened this issue Nov 13, 2016 · 1 comment

Comments

@KoyoSE
Copy link
Contributor

KoyoSE commented Nov 13, 2016

Current Behavior

lineTension option doesn't work for Radar chart type.
tension work.

radar chart linetention not work

Setting of this chart is lineTension = 0.5

Possible Solution

With this change, I think that this issue can be fixed.

https://github.com/chartjs/Chart.js/blob/master/src/controllers/controller.radar.js#L98

tension: custom.tension ? custom.tension : helpers.getValueOrDefault(dataset.lineTension, ....

Steps to Reproduce (for bugs)

jsfiddle

Environment

Chart.js version : 2.4.0
Browser name and version : Microsoft Edge 38.14393.0.0
Link to my project : https://jsfiddle.net/wzvhcpfy/2/

@etimberg
Copy link
Member

@KoyoSE lineTension is the correct property to use. It looks like the line chart has some code to port the old tension option.

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

2 participants