[PR #3855] [MERGED] (dashboards) piecharts - sorting and label spacing #31014

Closed
opened 2026-04-18 07:18:17 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3855
Author: @MatissJanis
Created: 11/19/2024
Status: Merged
Merged: 12/7/2024
Merged by: @MatissJanis

Base: masterHead: matiss/dashboards-pechart


📝 Commits (10+)

  • d969ba2 (dashboards) piecharts - sorting and label spacing
  • c6d810f Rename 3854.md to 3855.md
  • 3b46602 Update VRT
  • 77c3465 Update DonutGraph.tsx
  • e287049 Remove useMemo
  • 193ee8a Sorting patch
  • 8202aa5 Slice/sort
  • 1875f86 Merge branch 'master' into matiss/dashboards-pechart
  • cdabda2 Improve scaling of donut graph
  • d365c92 Merge branch 'master' into matiss/dashboards-pechart

📊 Changes

6 files changed (+106 additions, -70 deletions)

View changed files

📝 packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/reports/ChooseGraph.tsx (+0 -1)
📝 packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx (+100 -69)
upcoming-release-notes/3855.md (+6 -0)

📄 Description

Two changes:

  1. the piechart data will now be sorted (from biggest balance to smallest);
  2. slightly increased the spacing between the labels in the piechart (for mobile & dashboard page) - to give it a bit more breathing room and so it wouldn't intersect with the piechart itself

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/actualbudget/actual/pull/3855 **Author:** [@MatissJanis](https://github.com/MatissJanis) **Created:** 11/19/2024 **Status:** ✅ Merged **Merged:** 12/7/2024 **Merged by:** [@MatissJanis](https://github.com/MatissJanis) **Base:** `master` ← **Head:** `matiss/dashboards-pechart` --- ### 📝 Commits (10+) - [`d969ba2`](https://github.com/actualbudget/actual/commit/d969ba2c1fb9275de1bf12e7348bfc81b50f3ced) :sparkles: (dashboards) piecharts - sorting and label spacing - [`c6d810f`](https://github.com/actualbudget/actual/commit/c6d810f44dfb1d37a94f645be496e4589488fe95) Rename 3854.md to 3855.md - [`3b46602`](https://github.com/actualbudget/actual/commit/3b46602f54259bb9453ace5be0de1a0fe0f25222) Update VRT - [`77c3465`](https://github.com/actualbudget/actual/commit/77c3465dc9cf8d1f139b56f5baa5aac5c1668aa1) Update DonutGraph.tsx - [`e287049`](https://github.com/actualbudget/actual/commit/e2870493df2a3c834bbe352fbaa1d3b34ae1b235) Remove useMemo - [`193ee8a`](https://github.com/actualbudget/actual/commit/193ee8a708434d3dcb410d00da474d37220dc58f) Sorting patch - [`8202aa5`](https://github.com/actualbudget/actual/commit/8202aa52543f8fca064e3b835a834ab1b39a6ebd) Slice/sort - [`1875f86`](https://github.com/actualbudget/actual/commit/1875f86084a722ea84e72d449507eebf43f611d7) Merge branch 'master' into matiss/dashboards-pechart - [`cdabda2`](https://github.com/actualbudget/actual/commit/cdabda25cbb5d2740633f1942fc79ad9840f8f4f) Improve scaling of donut graph - [`d365c92`](https://github.com/actualbudget/actual/commit/d365c92bb9c61f611c5ddc3b6073ef59b9b816ac) Merge branch 'master' into matiss/dashboards-pechart ### 📊 Changes **6 files changed** (+106 additions, -70 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/reports.test.js-snapshots/Reports-custom-reports-Switches-to-Donut-Graph-and-checks-the-visuals-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/reports/ChooseGraph.tsx` (+0 -1) 📝 `packages/desktop-client/src/components/reports/graphs/DonutGraph.tsx` (+100 -69) ➕ `upcoming-release-notes/3855.md` (+6 -0) </details> ### 📄 Description Two changes: 1. the piechart data will now be sorted (from biggest balance to smallest); 2. slightly increased the spacing between the labels in the piechart (for mobile & dashboard page) - to give it a bit more breathing room and so it wouldn't intersect with the piechart itself --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-18 07:18:17 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#31014