Skip to content

Commit

Permalink
Fixes bank sync notification overlap with uncategorized transactions
Browse files Browse the repository at this point in the history
  • Loading branch information
The-Firexx committed Oct 5, 2024
1 parent a28fb93 commit 3e32b93
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 11 deletions.
15 changes: 6 additions & 9 deletions packages/desktop-client/src/components/BankSyncStatus.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -21,23 +21,20 @@ export function BankSyncStatus() {
const transitions = useTransition(
accountsSyncingCount > 0 ? 'syncing' : null,
{
from: { opacity: 0, transform: 'translateY(-100px)' },
enter: { opacity: 1, transform: 'translateY(0)' },
leave: { opacity: 0, transform: 'translateY(-100px)' },
from: { opacity: 0 },
enter: { opacity: 1 },
leave: { opacity: 0 },
unique: true,
},
);

return (
<View
style={{
position: 'absolute',
top: 0,
left: 0,
right: 0,
marginTop: 5,
alignItems: 'center',
zIndex: 501,
flexDirection: 'row',
overflow: 'hidden',
textWrap: 'nowrap',
}}
>
{transitions(
Expand Down
2 changes: 0 additions & 2 deletions packages/desktop-client/src/components/FinancesApp.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ import { useResponsive } from '../ResponsiveProvider';
import { theme } from '../style';
import { getIsOutdated, getLatestVersion } from '../util/versions';

import { BankSyncStatus } from './BankSyncStatus';
import { View } from './common/View';
import { GlobalKeys } from './GlobalKeys';
import { ManageRulesPage } from './ManageRulesPage';
Expand Down Expand Up @@ -174,7 +173,6 @@ export function FinancesApp() {
}}
/>
<Notifications />
<BankSyncStatus />

<Routes>
<Route
Expand Down
2 changes: 2 additions & 0 deletions packages/desktop-client/src/components/Titlebar.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import { theme, type CSSProperties, styles } from '../style';

import { AccountSyncCheck } from './accounts/AccountSyncCheck';
import { AnimatedRefresh } from './AnimatedRefresh';
import { BankSyncStatus } from './BankSyncStatus';
import { MonthCountSelector } from './budget/MonthCountSelector';
import { Button } from './common/Button2';
import { Link } from './common/Link';
Expand Down Expand Up @@ -332,6 +333,7 @@ export function Titlebar({ style }: TitlebarProps) {
<Route path="*" element={null} />
</Routes>
<View style={{ flex: 1 }} />
<BankSyncStatus />
<UncategorizedButton />
{isDevelopmentEnvironment() && !Platform.isPlaywright && (
<ThemeSelector style={{ marginLeft: 10 }} />
Expand Down
6 changes: 6 additions & 0 deletions upcoming-release-notes/3577.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
category: Bugfix
authors: [The-Firexx]
---

Fixes bank sync notification overlap with uncategorized transactions

0 comments on commit 3e32b93

Please sign in to comment.