[PR #6672] make colours on reports page consistent #6661

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

Original Pull Request: https://github.com/actualbudget/actual/pull/6672

State: closed
Merged: Yes


Addresses https://discord.com/channels/937901803608096828/1461442245570855022/1461442245570855022

I've also separated chartFill out, used for filling default charts. Using green makes sense, and as it inherits from positiveColour it will remain inline unless specifically overridden. I did it to let us use blue or "Actual Purple" for net worth for example.

Before:
image

After (green):
image

After (blue):
image

After (Actual purple):
image

Split chartFill colour (as an example):
image


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 14.4 MB → 14.4 MB (+461 B) +0.00%
loot-core 1 5.84 MB 0%
api 1 4.38 MB 0%
View detailed bundle stats

desktop-client

Total

Files count Total bundle size % Changed
28 14.4 MB → 14.4 MB (+461 B) +0.00%
Changeset
File Δ Size
src/components/reports/spreadsheets/calculateLegend.ts 📈 +261 B (+23.75%) 1.07 kB → 1.33 kB
src/components/reports/SummaryNumber.tsx 📈 +87 B (+3.82%) 2.22 kB → 2.31 kB
src/style/themes/development.ts 📈 +189 B (+2.34%) 7.88 kB → 8.07 kB
src/style/themes/dark.ts 📈 +189 B (+2.33%) 7.91 kB → 8.09 kB
src/style/themes/light.ts 📈 +189 B (+2.32%) 7.97 kB → 8.16 kB
src/style/themes/midnight.ts 📈 +173 B (+2.22%) 7.6 kB → 7.77 kB
home/runner/work/actual/actual/packages/component-library/src/theme.ts 📈 +240 B (+2.05%) 11.44 kB → 11.68 kB
src/components/reports/reports/SpendingCard.tsx 📈 +61 B (+0.74%) 8.01 kB → 8.07 kB
src/components/reports/graphs/CrossoverGraph.tsx 📈 +32 B (+0.44%) 7.1 kB → 7.13 kB
src/components/reports/graphs/AreaGraph.tsx 📈 +42 B (+0.34%) 12.14 kB → 12.18 kB
src/components/reports/graphs/CalendarGraph.tsx 📈 +33 B (+0.31%) 10.48 kB → 10.51 kB
src/components/reports/graphs/tableGraph/ReportTableRow.tsx 📈 +43 B (+0.31%) 13.73 kB → 13.77 kB
src/components/reports/graphs/NetWorthGraph.tsx 📈 +21 B (+0.29%) 7.05 kB → 7.07 kB
src/components/reports/reports/Summary.tsx 📈 +67 B (+0.24%) 27.29 kB → 27.35 kB
src/components/reports/reports/CalendarCard.tsx 📈 +22 B (+0.17%) 12.65 kB → 12.67 kB
src/components/reports/graphs/SpendingGraph.tsx 📈 +12 B (+0.13%) 8.95 kB → 8.96 kB
src/components/reports/graphs/CashFlowGraph.tsx 📈 +11 B (+0.11%) 9.39 kB → 9.4 kB
src/components/reports/reports/CashFlowCard.tsx 📈 +11 B (+0.10%) 11.28 kB → 11.29 kB
src/components/reports/FormulaResult.tsx 📉 -6 B (-0.16%) 3.62 kB → 3.61 kB
src/components/reports/reports/Calendar.tsx 📉 -103 B (-0.35%) 29.08 kB → 28.98 kB
src/components/reports/Change.tsx 📉 -204 B (-17.32%) 1.15 kB → 974 B
src/components/reports/chart-theme.ts 📉 -909 B (-36.74%) 2.42 kB → 1.53 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/ReportRouter.js 1.05 MB → 1.05 MB (+390 B) +0.04%
static/js/index.js 9.24 MB → 9.24 MB (+71 B) +0.00%

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
static/js/indexeddb-main-thread-worker-e59fee74.js 12.94 kB 0%
static/js/workbox-window.prod.es5.js 5.64 kB 0%
static/js/da.js 106.62 kB 0%
static/js/de.js 176.28 kB 0%
static/js/en-GB.js 6.98 kB 0%
static/js/en.js 159.37 kB 0%
static/js/es.js 171.98 kB 0%
static/js/fr.js 180.63 kB 0%
static/js/it.js 172.42 kB 0%
static/js/nb-NO.js 158.09 kB 0%
static/js/nl.js 103.35 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 146.32 kB 0%
static/js/ru.js 106.97 kB 0%
static/js/sv.js 78.28 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 216.34 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/narrow.js 640.86 kB 0%
static/js/TransactionList.js 101.58 kB 0%
static/js/wide.js 159.96 kB 0%
static/js/AppliedFilters.js 9.71 kB 0%
static/js/usePayeeRuleCounts.js 11.79 kB 0%
static/js/useTransactionBatchActions.js 13.23 kB 0%
static/js/FormulaEditor.js 1.04 MB 0%

loot-core

Total

Files count Total bundle size % Changed
1 5.84 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
kcab.worker.BInehE-N.js 5.84 MB 0%

api

Total

Files count Total bundle size % Changed
1 4.38 MB 0%
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger
No assets were bigger

Smaller
No assets were smaller

Unchanged

Asset File Size % Changed
bundle.api.js 4.38 MB 0%
**Original Pull Request:** https://github.com/actualbudget/actual/pull/6672 **State:** closed **Merged:** Yes --- Addresses https://discord.com/channels/937901803608096828/1461442245570855022/1461442245570855022 I've also separated chartFill out, used for filling default charts. Using green makes sense, and as it inherits from positiveColour it will remain inline unless specifically overridden. I did it to let us use blue or "Actual Purple" for net worth for example. Before: <img width="991" height="931" alt="image" src="https://github.com/user-attachments/assets/00a475fa-8844-4f77-961d-f5282a43e651" /> After (green): <img width="989" height="945" alt="image" src="https://github.com/user-attachments/assets/21df3eea-8346-4939-be04-64fbdcf69cdc" /> After (blue): <img width="985" height="935" alt="image" src="https://github.com/user-attachments/assets/46f9bcf9-c96f-4144-8fc7-9406767c050f" /> After (Actual purple): <img width="739" height="939" alt="image" src="https://github.com/user-attachments/assets/e8ed81c5-6f0d-4661-8af5-96e2a6a0c704" /> Split chartFill colour (as an example): <img width="727" height="919" alt="image" src="https://github.com/user-attachments/assets/4d2aec1d-7035-4048-8404-7d1fd2f2af7b" /> <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 28 | 14.4 MB → 14.4 MB (+461 B) | +0.00% loot-core | 1 | 5.84 MB | 0% api | 1 | 4.38 MB | 0% <details> <summary>View detailed bundle stats</summary> #### desktop-client **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 28 | 14.4 MB → 14.4 MB (+461 B) | +0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `src/components/reports/spreadsheets/calculateLegend.ts` | 📈 +261 B (+23.75%) | 1.07 kB → 1.33 kB `src/components/reports/SummaryNumber.tsx` | 📈 +87 B (+3.82%) | 2.22 kB → 2.31 kB `src/style/themes/development.ts` | 📈 +189 B (+2.34%) | 7.88 kB → 8.07 kB `src/style/themes/dark.ts` | 📈 +189 B (+2.33%) | 7.91 kB → 8.09 kB `src/style/themes/light.ts` | 📈 +189 B (+2.32%) | 7.97 kB → 8.16 kB `src/style/themes/midnight.ts` | 📈 +173 B (+2.22%) | 7.6 kB → 7.77 kB `home/runner/work/actual/actual/packages/component-library/src/theme.ts` | 📈 +240 B (+2.05%) | 11.44 kB → 11.68 kB `src/components/reports/reports/SpendingCard.tsx` | 📈 +61 B (+0.74%) | 8.01 kB → 8.07 kB `src/components/reports/graphs/CrossoverGraph.tsx` | 📈 +32 B (+0.44%) | 7.1 kB → 7.13 kB `src/components/reports/graphs/AreaGraph.tsx` | 📈 +42 B (+0.34%) | 12.14 kB → 12.18 kB `src/components/reports/graphs/CalendarGraph.tsx` | 📈 +33 B (+0.31%) | 10.48 kB → 10.51 kB `src/components/reports/graphs/tableGraph/ReportTableRow.tsx` | 📈 +43 B (+0.31%) | 13.73 kB → 13.77 kB `src/components/reports/graphs/NetWorthGraph.tsx` | 📈 +21 B (+0.29%) | 7.05 kB → 7.07 kB `src/components/reports/reports/Summary.tsx` | 📈 +67 B (+0.24%) | 27.29 kB → 27.35 kB `src/components/reports/reports/CalendarCard.tsx` | 📈 +22 B (+0.17%) | 12.65 kB → 12.67 kB `src/components/reports/graphs/SpendingGraph.tsx` | 📈 +12 B (+0.13%) | 8.95 kB → 8.96 kB `src/components/reports/graphs/CashFlowGraph.tsx` | 📈 +11 B (+0.11%) | 9.39 kB → 9.4 kB `src/components/reports/reports/CashFlowCard.tsx` | 📈 +11 B (+0.10%) | 11.28 kB → 11.29 kB `src/components/reports/FormulaResult.tsx` | 📉 -6 B (-0.16%) | 3.62 kB → 3.61 kB `src/components/reports/reports/Calendar.tsx` | 📉 -103 B (-0.35%) | 29.08 kB → 28.98 kB `src/components/reports/Change.tsx` | 📉 -204 B (-17.32%) | 1.15 kB → 974 B `src/components/reports/chart-theme.ts` | 📉 -909 B (-36.74%) | 2.42 kB → 1.53 kB </details> <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** Asset | File Size | % Changed ----- | --------- | --------- static/js/ReportRouter.js | 1.05 MB → 1.05 MB (+390 B) | +0.04% static/js/index.js | 9.24 MB → 9.24 MB (+71 B) | +0.00% **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- static/js/indexeddb-main-thread-worker-e59fee74.js | 12.94 kB | 0% static/js/workbox-window.prod.es5.js | 5.64 kB | 0% static/js/da.js | 106.62 kB | 0% static/js/de.js | 176.28 kB | 0% static/js/en-GB.js | 6.98 kB | 0% static/js/en.js | 159.37 kB | 0% static/js/es.js | 171.98 kB | 0% static/js/fr.js | 180.63 kB | 0% static/js/it.js | 172.42 kB | 0% static/js/nb-NO.js | 158.09 kB | 0% static/js/nl.js | 103.35 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 146.32 kB | 0% static/js/ru.js | 106.97 kB | 0% static/js/sv.js | 78.28 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 216.34 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/narrow.js | 640.86 kB | 0% static/js/TransactionList.js | 101.58 kB | 0% static/js/wide.js | 159.96 kB | 0% static/js/AppliedFilters.js | 9.71 kB | 0% static/js/usePayeeRuleCounts.js | 11.79 kB | 0% static/js/useTransactionBatchActions.js | 13.23 kB | 0% static/js/FormulaEditor.js | 1.04 MB | 0% </div> </details> --- #### loot-core **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 5.84 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- kcab.worker.BInehE-N.js | 5.84 MB | 0% </div> </details> --- #### api **Total** Files count | Total bundle size | % Changed ----------- | ----------------- | --------- 1 | 4.38 MB | 0% <details> <summary>View detailed bundle breakdown</summary> <div> **Added** No assets were added **Removed** No assets were removed **Bigger** No assets were bigger **Smaller** No assets were smaller **Unchanged** Asset | File Size | % Changed ----- | --------- | --------- bundle.api.js | 4.38 MB | 0% </div> </details> </details> <!--- bundlestats-action-comment key:combined end --->
GiteaMirror added the pull-request label 2026-02-28 21:31:12 -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#6661