Feature/radar chart vertice label #1914
Conversation
|
I didn't get what the difference between |
|
And your sample looks beautiful. Can you please add it in our samples list? |
|
Thank you for your feedback! After reconsidering, I realized that The role of
I will also add the sample to the samples list as you suggested! |
|
@imaNNeo |
| final TextStyle? titleTextStyle; | ||
|
|
||
| /// Defines style of showing [RadarChart] vertice labels. | ||
| final TextStyle? verticeLabelTextStyle; |
There was a problem hiding this comment.
It seems you forgot to add this verticeLabelTextStyle property in the copyWith and lerp functions
| } | ||
|
|
||
| /// Defines a label for [RadarChart] vertices | ||
| class RadarChartVerticeLabel { |
There was a problem hiding this comment.
There's a typo here, and this is what I've got from the Chatgpt:
| /// [positionPercentageOffset] is the place of showing label on the [RadarChart] vertices | ||
| /// The higher the value of this field, the more labels move away from the chart vertices. | ||
| /// This value should be between 0 and 1 | ||
| final double? positionPercentageOffset; |
There was a problem hiding this comment.
Can we just add a final double offset; here? I mean it makes it much simpler, as we know where it draws (at the position of our vertex point). So we can just have a simple offest property
2 participants
Summary
Added a new feature to display labels at each vertex of the RadarChart.
Fixes this issue: #1886
What's included
New Classes and Methods
RadarChartVerticeLabelclass to handle vertex label configurationtext: Displays the label textpositionPercentageOffset: Controls label position relative to vertexTests
Added comprehensive test cases for
drawVerticeLabels:Screenshot