[PR #5104] [MERGED] Add colors to ReportTable "Totals" row #5758

Closed
opened 2026-02-28 21:18:03 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/5104
Author: @pogman-code
Created: 6/5/2025
Status: Merged
Merged: 7/21/2025
Merged by: @matt-fidd

Base: masterHead: report-table-colors


📝 Commits (3)

📊 Changes

7 files changed (+27 additions, -2 deletions)

View changed files

📝 packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTable.tsx (+1 -0)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableRow.tsx (+18 -0)
📝 packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx (+2 -2)
upcoming-release-notes/5104.md (+6 -0)

📄 Description

Most reports use colors to make data more readable but ReportTable has a lot of data but not color to make important data pop more.

I have added colors to the "Totals" row
image
image

What do you guys think?


🔄 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/5104 **Author:** [@pogman-code](https://github.com/pogman-code) **Created:** 6/5/2025 **Status:** ✅ Merged **Merged:** 7/21/2025 **Merged by:** [@matt-fidd](https://github.com/matt-fidd) **Base:** `master` ← **Head:** `report-table-colors` --- ### 📝 Commits (3) - [`5271d52`](https://github.com/actualbudget/actual/commit/5271d5269fcf01d6ff41766c20bca52ef8c369b6) add colors to ReportTable - [`50f4821`](https://github.com/actualbudget/actual/commit/50f482145b761d3ec5bbe585cc0a3379498d4e72) Update VRT - [`1cdc852`](https://github.com/actualbudget/actual/commit/1cdc852de1a786fb6b4f43b6d348c3b1ff2a9b36) use default color for zero values ### 📊 Changes **7 files changed** (+27 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/reports.test.ts-snapshots/Reports-custom-reports-Switches-to-Data-Table-and-checks-the-visuals-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTable.tsx` (+1 -0) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableRow.tsx` (+18 -0) 📝 `packages/desktop-client/src/components/reports/graphs/tableGraph/ReportTableTotals.tsx` (+2 -2) ➕ `upcoming-release-notes/5104.md` (+6 -0) </details> ### 📄 Description Most reports use colors to make data more readable but ReportTable has a lot of data but not color to make important data pop more. I have added colors to the "Totals" row ![image](https://github.com/user-attachments/assets/d47d920b-125a-4fe8-b631-edb85794b535) ![image](https://github.com/user-attachments/assets/9c0a66ea-e2c0-4569-8c43-bec225dfc1de) What do you guys think? --- <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-02-28 21:18:03 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#5758