[PR #6764] [MERGED] make colors on budget page more legibile #41154

Closed
opened 2026-04-23 13:57:11 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6764
Author: @matt-fidd
Created: 1/23/2026
Status: Merged
Merged: 1/24/2026
Merged by: @youngcw

Base: masterHead: colours/fix


📝 Commits (10+)

📊 Changes

18 files changed (+28 additions, -17 deletions)

View changed files

📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-1-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-2-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-3-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-4-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-5-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-6-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-7-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-8-chromium-linux.png (+0 -0)
📝 packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-9-chromium-linux.png (+0 -0)
📝 packages/desktop-client/src/components/budget/envelope/EnvelopeBudgetComponents.tsx (+2 -2)
📝 packages/desktop-client/src/components/mobile/budget/BudgetTable.tsx (+4 -4)
📝 packages/desktop-client/src/components/mobile/transactions/TransactionListWithBalances.tsx (+5 -1)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.tsx (+2 -1)
📝 packages/desktop-client/src/style/themes/dark.ts (+2 -2)
📝 packages/desktop-client/src/style/themes/development.ts (+3 -3)
📝 packages/desktop-client/src/style/themes/light.ts (+1 -1)
📝 packages/desktop-client/src/style/themes/midnight.ts (+3 -3)
upcoming-release-notes/6764.md (+6 -0)

📄 Description

The new colours were causing issues on Pixel phones, also caught a few more missed number based colours too.


Bundle Stats

Bundle Files count Total bundle size % Changed
desktop-client 28 14.42 MB → 14.42 MB (+94 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.42 MB → 14.42 MB (+94 B) +0.00%
Changeset
File Δ Size
src/components/mobile/transactions/TransactionListWithBalances.tsx 📈 +40 B (+0.37%) 10.59 kB → 10.63 kB
src/components/mobile/budget/BudgetTable.tsx 📈 +25 B (+0.10%) 24.3 kB → 24.32 kB
src/components/budget/envelope/EnvelopeBudgetComponents.tsx 📈 +25 B (+0.08%) 29.98 kB → 30.01 kB
src/components/transactions/TransactionsTable.tsx 📈 +4 B (+0.00%) 85.88 kB → 85.88 kB
View detailed bundle breakdown

Added
No assets were added

Removed
No assets were removed

Bigger

Asset File Size % Changed
static/js/narrow.js 640.87 kB → 640.94 kB (+65 B) +0.01%
static/js/index.js 9.24 MB → 9.24 MB (+29 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 175.37 kB 0%
static/js/en-GB.js 7.18 kB 0%
static/js/en.js 160.05 kB 0%
static/js/es.js 171.21 kB 0%
static/js/fr.js 179.72 kB 0%
static/js/it.js 171.54 kB 0%
static/js/nb-NO.js 157.23 kB 0%
static/js/nl.js 103.35 kB 0%
static/js/pl.js 88.64 kB 0%
static/js/pt-BR.js 146.24 kB 0%
static/js/ru.js 106.97 kB 0%
static/js/sv.js 78.2 kB 0%
static/js/th.js 182.35 kB 0%
static/js/uk.js 215.11 kB 0%
static/js/resize-observer.js 18.37 kB 0%
static/js/BackgroundImage.js 120.54 kB 0%
static/js/ReportRouter.js 1.07 MB 0%
static/js/TransactionList.js 101.58 kB 0%
static/js/wide.js 159.97 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%

🔄 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/6764 **Author:** [@matt-fidd](https://github.com/matt-fidd) **Created:** 1/23/2026 **Status:** ✅ Merged **Merged:** 1/24/2026 **Merged by:** [@youngcw](https://github.com/youngcw) **Base:** `master` ← **Head:** `colours/fix` --- ### 📝 Commits (10+) - [`31a16a7`](https://github.com/actualbudget/actual/commit/31a16a7f648b160f7183550a666e9e07502ea39a) use old budget page colours - [`dfd9a05`](https://github.com/actualbudget/actual/commit/dfd9a050a8131348926b84b9aa17f9e7d97f43c2) running balance - [`4d183c4`](https://github.com/actualbudget/actual/commit/4d183c43badc86e3e8beed979256ff3b68721b52) schedule indicator - [`3db4be6`](https://github.com/actualbudget/actual/commit/3db4be6674b2d799e0ff08084f8b9fe991c18bdf) mobile to budget - [`344fad3`](https://github.com/actualbudget/actual/commit/344fad3ca72f40c0bf54c4b62ec53931a69c1d3f) note - [`a4d345a`](https://github.com/actualbudget/actual/commit/a4d345a8356da3ed6f69ea5a51a2cf5324d40520) template indicator due should match underfunded - [`513300f`](https://github.com/actualbudget/actual/commit/513300fb495d4aef5b4f9216f9fb988d2f13b788) coderabbit - [`232c28a`](https://github.com/actualbudget/actual/commit/232c28a895136726ceec84977f658fb0ea653775) mobile to budget neutral zero - [`080a095`](https://github.com/actualbudget/actual/commit/080a095bf4963fb317ce887110ee51085e0d20c5) matiss feedback - [`ba4e6b7`](https://github.com/actualbudget/actual/commit/ba4e6b74e4b96b910ba81988f852ec7a0a03f543) mobile account balance ### 📊 Changes **18 files changed** (+28 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-1-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-2-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-3-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-4-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-5-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-6-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-7-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-8-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/e2e/accounts.mobile.test.ts-snapshots/Mobile-Accounts-opens-individual-account-page-and-checks-that-filtering-is-working-9-chromium-linux.png` (+0 -0) 📝 `packages/desktop-client/src/components/budget/envelope/EnvelopeBudgetComponents.tsx` (+2 -2) 📝 `packages/desktop-client/src/components/mobile/budget/BudgetTable.tsx` (+4 -4) 📝 `packages/desktop-client/src/components/mobile/transactions/TransactionListWithBalances.tsx` (+5 -1) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.tsx` (+2 -1) 📝 `packages/desktop-client/src/style/themes/dark.ts` (+2 -2) 📝 `packages/desktop-client/src/style/themes/development.ts` (+3 -3) 📝 `packages/desktop-client/src/style/themes/light.ts` (+1 -1) 📝 `packages/desktop-client/src/style/themes/midnight.ts` (+3 -3) ➕ `upcoming-release-notes/6764.md` (+6 -0) </details> ### 📄 Description The new colours were causing issues on Pixel phones, also caught a few more missed number based colours too. <!--- actual-bot-sections ---> <hr /> <!--- bundlestats-action-comment key:combined start ---> ### Bundle Stats Bundle | Files count | Total bundle size | % Changed ------ | ----------- | ----------------- | --------- desktop-client | 28 | 14.42 MB → 14.42 MB (+94 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.42 MB → 14.42 MB (+94 B) | +0.00% <details> <summary>Changeset</summary> File | Δ | Size ---- | - | ---- `src/components/mobile/transactions/TransactionListWithBalances.tsx` | 📈 +40 B (+0.37%) | 10.59 kB → 10.63 kB `src/components/mobile/budget/BudgetTable.tsx` | 📈 +25 B (+0.10%) | 24.3 kB → 24.32 kB `src/components/budget/envelope/EnvelopeBudgetComponents.tsx` | 📈 +25 B (+0.08%) | 29.98 kB → 30.01 kB `src/components/transactions/TransactionsTable.tsx` | 📈 +4 B (+0.00%) | 85.88 kB → 85.88 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/narrow.js | 640.87 kB → 640.94 kB (+65 B) | +0.01% static/js/index.js | 9.24 MB → 9.24 MB (+29 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 | 175.37 kB | 0% static/js/en-GB.js | 7.18 kB | 0% static/js/en.js | 160.05 kB | 0% static/js/es.js | 171.21 kB | 0% static/js/fr.js | 179.72 kB | 0% static/js/it.js | 171.54 kB | 0% static/js/nb-NO.js | 157.23 kB | 0% static/js/nl.js | 103.35 kB | 0% static/js/pl.js | 88.64 kB | 0% static/js/pt-BR.js | 146.24 kB | 0% static/js/ru.js | 106.97 kB | 0% static/js/sv.js | 78.2 kB | 0% static/js/th.js | 182.35 kB | 0% static/js/uk.js | 215.11 kB | 0% static/js/resize-observer.js | 18.37 kB | 0% static/js/BackgroundImage.js | 120.54 kB | 0% static/js/ReportRouter.js | 1.07 MB | 0% static/js/TransactionList.js | 101.58 kB | 0% static/js/wide.js | 159.97 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 ---> --- <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-23 13:57:11 -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#41154