[GH-ISSUE #7671] [Bug]: Titlebar server sync icon not centred #72895

Closed
opened 2026-05-16 13:49:04 -05:00 by GiteaMirror · 2 comments
Owner

Originally created by @tjmills-dev on GitHub (May 1, 2026).
Original GitHub issue: https://github.com/actualbudget/actual/issues/7671

What happened?

The server sync icon in the titlebar is not centred in its div, unlike the eye icon:

Image

This results from an empty span with a 3px left margin.

Image

The offending line can be found at 1afe7c9a1e/packages/desktop-client/src/components/Titlebar.tsx (L246) - I'm not sure why the margin is added as the flexbox takes care of the spacing, so might be that it can just be removed?

How can we reproduce the issue?

  1. Go to any page on the web client that has server syncing
  2. Hover over the sync icon
  3. Notice that the icon is not quite centred to its shaded background
  4. Be unable to unnotice this

Where are you hosting Actual?

Docker

What browsers are you seeing the problem on?

Firefox

Operating System

Linux

Originally created by @tjmills-dev on GitHub (May 1, 2026). Original GitHub issue: https://github.com/actualbudget/actual/issues/7671 ### What happened? The server sync icon in the titlebar is not centred in its div, unlike the eye icon: <img width="154" height="117" alt="Image" src="https://github.com/user-attachments/assets/f09d4cb5-227e-4311-9d31-851e80639f14" /> This results from an empty span with a 3px left margin. <img width="426" height="157" alt="Image" src="https://github.com/user-attachments/assets/6a628835-3b15-4b82-a75f-719d7bb48ae0" /> The offending line can be found at https://github.com/actualbudget/actual/blob/1afe7c9a1e55fb76e3663f14edfd6dd6db56e4bc/packages/desktop-client/src/components/Titlebar.tsx#L246 - I'm not sure why the margin is added as the flexbox takes care of the spacing, so might be that it can just be removed? ### How can we reproduce the issue? 1. Go to any page on the web client that has server syncing 2. Hover over the sync icon 3. Notice that the icon is not quite centred to its shaded background 4. Be unable to unnotice this ### Where are you hosting Actual? Docker ### What browsers are you seeing the problem on? Firefox ### Operating System Linux
GiteaMirror added the bug label 2026-05-16 13:49:04 -05:00
Author
Owner

@Juulz commented on GitHub (May 1, 2026):

The text is null unless it is 'Disabled', so it may not be empty. I suppose the text styling could be in the syncState === 'disabled' block.

I will fix this. Thanks for the catch!

<!-- gh-comment-id:4359451323 --> @Juulz commented on GitHub (May 1, 2026): The text is null unless it is 'Disabled', so it may not be empty. I suppose the text styling could be in the `syncState === 'disabled'` block. I will fix this. Thanks for the catch!
Author
Owner

@tjmills-dev commented on GitHub (May 1, 2026):

Ah yes I see now. Thanks for the fix!

<!-- gh-comment-id:4362113770 --> @tjmills-dev commented on GitHub (May 1, 2026): Ah yes I see now. Thanks for the fix!
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/actual#72895