[PR #6397] [CLOSED] Fix selected balance not showing when sum is $0 #55796

Closed
opened 2026-05-01 03:05:16 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/6397
Author: @kellyarwine
Created: 12/13/2025
Status: Closed

Base: masterHead: fix-zero-selected-balance


📝 Commits (2)

  • 90de50f Fix selected balance not showing when sum is $0
  • 60f7d7e Merge branch 'master' into fix-zero-selected-balance

📊 Changes

5 files changed (+420 additions, -63 deletions)

View changed files

packages/desktop-client/src/components/accounts/Balance.test.tsx (+286 -0)
📝 packages/desktop-client/src/components/accounts/Balance.tsx (+5 -2)
📝 packages/desktop-client/src/components/transactions/TransactionsTable.tsx (+98 -61)
📝 packages/desktop-client/src/setupTests.js (+25 -0)
upcoming-release-notes/6397.md (+6 -0)

📄 Description

👋 Hi, AB team! This is my first PR contributing to this project.

image

Changes

The condition used a falsy check which treated 0 the same as null, causing the selected balance to hide when transactions summed to zero.

Also fixes a pre-existing setState-during-render issue in TransactionsTable.


🔄 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/6397 **Author:** [@kellyarwine](https://github.com/kellyarwine) **Created:** 12/13/2025 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix-zero-selected-balance` --- ### 📝 Commits (2) - [`90de50f`](https://github.com/actualbudget/actual/commit/90de50ff609fc20f75dd97a57b58ebaba81e0e34) Fix selected balance not showing when sum is $0 - [`60f7d7e`](https://github.com/actualbudget/actual/commit/60f7d7eb3cdfc09127c30c4493ebd276dd07f806) Merge branch 'master' into fix-zero-selected-balance ### 📊 Changes **5 files changed** (+420 additions, -63 deletions) <details> <summary>View changed files</summary> ➕ `packages/desktop-client/src/components/accounts/Balance.test.tsx` (+286 -0) 📝 `packages/desktop-client/src/components/accounts/Balance.tsx` (+5 -2) 📝 `packages/desktop-client/src/components/transactions/TransactionsTable.tsx` (+98 -61) 📝 `packages/desktop-client/src/setupTests.js` (+25 -0) ➕ `upcoming-release-notes/6397.md` (+6 -0) </details> ### 📄 Description 👋 Hi, AB team! This is my first PR contributing to this project. <img width="3420" height="1906" alt="image" src="https://github.com/user-attachments/assets/712133cd-cb82-4d1d-a0a9-e1fd08e5b8fd" /> ## Changes The condition used a falsy check which treated 0 the same as null, causing the selected balance to hide when transactions summed to zero. Also fixes a pre-existing setState-during-render issue in TransactionsTable. --- <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-05-01 03:05:16 -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#55796