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

Add Feature to Display Scores Beside Polygon Points in RadarChart #120

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

Develoferr
Copy link

Pull Request: Add Feature to Display Scores Beside Polygon Points in RadarChart
This PR introduces a new feature to enhance the RadarChart by displaying scores beside the polygon points.

Scores are rendered above the points in the upper half of the graph and below the points in the lower half, depending on their position.
The visibility of the scores can be controlled through a boolean variable named showScores, allowing users to toggle this feature as needed.
This enhancement aims to improve the visual clarity of the chart by presenting scores more effectively. This feature arises from a personal need to better visualize the data, particularly in an educational context where scores range from 1 to 10. I believe this addition will be beneficial for better visualizing data in such settings.

In my opinion, this PR makes the most sense when paired with the second PR (submitted separately), which allows scalar values to be deactivated. An alternative option would be to automatically disable scalar values when scores are enabled, for a cleaner visual presentation.

A potential improvement could involve calculating the score positions dynamically to avoid overlap when two scores occupy the same space; however, this approach would add considerable complexity to the code.

I remain attentive to any suggestions, questions, or comments you may have!

- Scores are displayed above the points in the upper half of the graph and below the points in the lower half.
- The feature can be activated by a boolean (showScores).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant