[PR #3578] [CLOSED] Fixes #3536 Fixes bank sync notification overlap with uncategorized transactions #4933

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3578
Author: @The-Firexx
Created: 10/5/2024
Status: Closed

Base: masterHead: fix-notification-overlap


📝 Commits (4)

  • 3e32b93 Fixes bank sync notification overlap with uncategorized transactions
  • f34b1c0 Renamed the release notes files
  • 72ae423 Merge branch 'master' into fix-notification-overlap
  • 9e9c5dd Merge branch 'master' into fix-notification-overlap

📊 Changes

4 files changed (+14 additions, -11 deletions)

View changed files

📝 packages/desktop-client/src/components/BankSyncStatus.tsx (+6 -9)
📝 packages/desktop-client/src/components/FinancesApp.tsx (+0 -2)
📝 packages/desktop-client/src/components/Titlebar.tsx (+2 -0)
upcoming-release-notes/3578.md (+6 -0)

📄 Description

This pull request is to avoid the bank sync notification overlap with uncategorized transactions.
This fixes #3536
Visually it will now look like this. The "Bank sync notification" will start to overflow as hidden.
I also changed the position of the div to stay in the same div as the rest of the buttons, losing is "center position" when the window is full resolution.

imagem

If the solution is not ideal, please close this pull request.
Thank you.


🔄 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/3578 **Author:** [@The-Firexx](https://github.com/The-Firexx) **Created:** 10/5/2024 **Status:** ❌ Closed **Base:** `master` ← **Head:** `fix-notification-overlap` --- ### 📝 Commits (4) - [`3e32b93`](https://github.com/actualbudget/actual/commit/3e32b930e3ded1a1889cde0797f8dda62e64610b) Fixes bank sync notification overlap with uncategorized transactions - [`f34b1c0`](https://github.com/actualbudget/actual/commit/f34b1c093e7b0d327ea9960defa5cf806a8889d3) Renamed the release notes files - [`72ae423`](https://github.com/actualbudget/actual/commit/72ae423782ffaf793d293818f78b81f41afaafdd) Merge branch 'master' into fix-notification-overlap - [`9e9c5dd`](https://github.com/actualbudget/actual/commit/9e9c5ddc60bc1e451957f62d71ce52be3e2283aa) Merge branch 'master' into fix-notification-overlap ### 📊 Changes **4 files changed** (+14 additions, -11 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/BankSyncStatus.tsx` (+6 -9) 📝 `packages/desktop-client/src/components/FinancesApp.tsx` (+0 -2) 📝 `packages/desktop-client/src/components/Titlebar.tsx` (+2 -0) ➕ `upcoming-release-notes/3578.md` (+6 -0) </details> ### 📄 Description This pull request is to avoid the bank sync notification overlap with uncategorized transactions. This fixes #3536 Visually it will now look like this. The "Bank sync notification" will start to overflow as hidden. I also changed the position of the div to stay in the same div as the rest of the buttons, losing is "center position" when the window is full resolution. ![imagem](https://github.com/user-attachments/assets/4f00fed1-7fe6-489a-8d5e-7975368b42cd) If the solution is not ideal, please close this pull request. Thank you. --- <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:03:26 -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#4933