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

Radar chart rendering issue #7444

Closed
stockiNail opened this issue May 30, 2020 · 4 comments · Fixed by #7446
Closed

Radar chart rendering issue #7444

stockiNail opened this issue May 30, 2020 · 4 comments · Fixed by #7446
Milestone

Comments

@stockiNail
Copy link
Contributor

stockiNail commented May 30, 2020

Expected Behavior

BE AWARE this issue was solved in 2.9.3 version (see the issue #6396).

The rendering of a radar chart works correctly with version 2.9.3.

Radar293

Current Behavior

Using the development build, the rendering of the chart will apply a "white" area in the middle of the chart itself.

Radar300

Steps to Reproduce (for bugs)

https://codepen.io/stockinail/pen/KKdOXRM

Environment

@stockiNail stockiNail changed the title Radar chart rendering issue (using "master" v3) Radar chart rendering issue May 30, 2020
@benmccann benmccann added this to the Version 3.0 milestone May 30, 2020
@etimberg
Copy link
Member

Adding fill: 'start' works around it. https://codepen.io/etimberg/pen/pojMazo

Will need to figure out what changed

@etimberg
Copy link
Member

v2.9 defaulted to fill: 'start'. Looking to see why that changed https://github.com/chartjs/Chart.js/blob/2.9/src/controllers/controller.radar.js#L17

@etimberg
Copy link
Member

Looks like we fixed the backwrard incompatibilty in #6655 but never PRed that to master. I will do that since the default is a lot better.

@stockiNail
Copy link
Contributor Author

Adding fill: 'start' works around it. https://codepen.io/etimberg/pen/pojMazo

Will need to figure out what changed

@etimberg Yes, I was aware about that as workaround.

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

Successfully merging a pull request may close this issue.

3 participants