[PR #3547] [CLOSED] Visually deconflict sync status from notifications button #4919

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

📋 Pull Request Information

Original PR: https://github.com/actualbudget/actual/pull/3547
Author: @afv22
Created: 10/2/2024
Status: Closed

Base: masterHead: master


📝 Commits (5)

  • aa93d79 Hover 'Syncing' animation over notification button
  • 4c9fb58 Add release note
  • 2c1f25e Center Syncing notification without blocking background elements
  • 2a1fb18 Merge branch 'actualbudget:master' into master
  • 4578404 Merge branch 'actualbudget:master' into master

📊 Changes

2 files changed (+10 additions, -3 deletions)

View changed files

📝 packages/desktop-client/src/components/BankSyncStatus.tsx (+4 -3)
upcoming-release-notes/3547.md (+6 -0)

📄 Description

Fixes #3536

Increasing stack placement of the "Syncing..." alert so it does not visually conflict with the notifications button.

Before

Screenshot 2024-10-02 at 5 42 27 PM

After

Screenshot 2024-10-02 at 5 42 39 PM

🔄 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/3547 **Author:** [@afv22](https://github.com/afv22) **Created:** 10/2/2024 **Status:** ❌ Closed **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (5) - [`aa93d79`](https://github.com/actualbudget/actual/commit/aa93d79e54d09e5b1a06fa908a3b35c108cdea5c) Hover 'Syncing' animation over notification button - [`4c9fb58`](https://github.com/actualbudget/actual/commit/4c9fb5864dbed0481f901db7d94a3630da9ad0ff) Add release note - [`2c1f25e`](https://github.com/actualbudget/actual/commit/2c1f25e8dd89fd8b41527aed44cb036c95ad3432) Center Syncing notification without blocking background elements - [`2a1fb18`](https://github.com/actualbudget/actual/commit/2a1fb18f71e0338e566bff9b42f37edc9d7623c9) Merge branch 'actualbudget:master' into master - [`4578404`](https://github.com/actualbudget/actual/commit/457840496862987d82760e5ffb4b00d60245feb1) Merge branch 'actualbudget:master' into master ### 📊 Changes **2 files changed** (+10 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `packages/desktop-client/src/components/BankSyncStatus.tsx` (+4 -3) ➕ `upcoming-release-notes/3547.md` (+6 -0) </details> ### 📄 Description Fixes #3536 Increasing stack placement of the "Syncing..." alert so it does not visually conflict with the notifications button. ## Before <img width="656" alt="Screenshot 2024-10-02 at 5 42 27 PM" src="https://github.com/user-attachments/assets/30fd74ee-cadb-4ec2-8038-49c6ce0dcdf4"> ## After <img width="654" alt="Screenshot 2024-10-02 at 5 42 39 PM" src="https://github.com/user-attachments/assets/89fe6dd0-d3cf-420b-bafe-c5a446dd2572"> --- <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:14 -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#4919