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.

Sign up for GitHub

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

Jump to bottom

Cannot set step size for radar chart #110

Closed
Joelius300 opened this issue May 12, 2020 · 0 comments · Fixed by #120
Closed

Cannot set step size for radar chart #110

Joelius300 opened this issue May 12, 2020 · 0 comments · Fixed by #120
Labels
bug Something isn't working

Comments

@Joelius300
Copy link
Contributor

This issues is based on #108.

Describe the bug

In the radar chart there's a Scale class with a CartesianTicks property. However, the radar chart isn't a cartesian chart. Instead it's a linear radial chart, like the polar chart. The Scale property should rather be of type LinearRadialAxis.

Which Blazor project type is your bug related to?

  • Client-Side
  • Server-Side

Which charts does this bug apply to?

Radar chart

Additional context / logging

This will be part of the much needed rework of the radar chart. PRs are welcome but probably won't be merged directly. If you consider submitting a PR, please respect our coding conventions and check out our other charts.

Code example

Provided internally and in #108.
@Joelius300 Joelius300 added the bug Something isn't working label May 12, 2020
@Joelius300 Joelius300 mentioned this issue May 12, 2020
Closed
Joelius300 added a commit to Joelius300/ChartJs.Blazor that referenced this issue Jun 2, 2020
@Joelius300
Fix radar-chart
c973bef 
- Scale changed to LinearRadialAxis
- Update summaries
- Fixes mariusmuntean#110
@Joelius300 Joelius300 mentioned this issue Jun 2, 2020
Merged
Joelius300 added a commit that referenced this issue Jun 3, 2020
@Joelius300
Fix radar-chart
6203279 
- Scale changed to LinearRadialAxis
- Update summaries
- Fixes #110
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix radar-chart Joelius300/ChartJs.Blazor
1 participant
@Joelius300