Skip to content

Conversation

@UmakanthKaspa
Copy link

Description:

This PR introduces a new feature for the Donut Chart, allowing users to display the total value of all segments in the center of the chart.
Closes #422

Before:

image

After:

image

@UmakanthKaspa UmakanthKaspa force-pushed the feature/donut-total-display branch 2 times, most recently from 35bc85e to b48c975 Compare January 5, 2025 07:05
@nextchamp-saqib
Copy link
Member

The center is off for other legend positions

CleanShot 2025-01-10 at 13 19 35@2x

Also, both "Total" & "Number" shouldn't be bold, maybe something like this, if possible

CleanShot 2025-01-10 at 13 22 21@2x

@UmakanthKaspa UmakanthKaspa force-pushed the feature/donut-total-display branch from 133e634 to 6e90bb9 Compare January 20, 2025 12:24
@UmakanthKaspa
Copy link
Author

The center is off for other legend positions

CleanShot 2025-01-10 at 13 19 35@2x

Also, both "Total" & "Number" shouldn't be bold, maybe something like this, if possible

CleanShot 2025-01-10 at 13 22 21@2x

@nextchamp-saqib I've centered the total. Please check.

@UmakanthKaspa UmakanthKaspa marked this pull request as draft May 17, 2025 11:12
@UmakanthKaspa UmakanthKaspa marked this pull request as ready for review July 8, 2025 03:51
@UmakanthKaspa UmakanthKaspa marked this pull request as draft July 8, 2025 03:52
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.

Add Total Value Display to Donut Chart

2 participants