Skip to content

Commit

Permalink
fixes alignment of notifications on mobile to be centered (actualbudg…
Browse files Browse the repository at this point in the history
  • Loading branch information
YusefOuda authored Jul 13, 2024
1 parent 32d8304 commit 942aebe
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 0 deletions.
3 changes: 3 additions & 0 deletions packages/desktop-client/src/components/Notifications.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -13,6 +13,7 @@ import type { NotificationWithId } from 'loot-core/src/client/state-types/notifi
import { useActions } from '../hooks/useActions';
import { AnimatedLoading } from '../icons/AnimatedLoading';
import { SvgDelete } from '../icons/v0';
import { useResponsive } from '../ResponsiveProvider';
import { styles, theme, type CSSProperties } from '../style';

import { Button, ButtonWithLoading } from './common/Button';
Expand Down Expand Up @@ -245,6 +246,7 @@ function Notification({

export function Notifications({ style }: { style?: CSSProperties }) {
const { removeNotification } = useActions();
const { isNarrowWidth } = useResponsive();
const notifications = useSelector(
(state: State) => state.notifications.notifications,
);
Expand All @@ -254,6 +256,7 @@ export function Notifications({ style }: { style?: CSSProperties }) {
position: 'fixed',
bottom: 20,
right: 13,
left: isNarrowWidth ? 13 : undefined,
zIndex: 10000,
...style,
}}
Expand Down
6 changes: 6 additions & 0 deletions upcoming-release-notes/3046.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
---
category: Bugfix
authors: [YusefOuda]
---

Fixes the alignment of notifications in mobile view

0 comments on commit 942aebe

Please sign in to comment.