fix: show notifications on mobile (#17008)

Show the notifications close to the menu button.

**After fix:**
<img width="515" alt="Screenshot 2025-03-19 at 15 09 36"
src="https://github.com/user-attachments/assets/38f86c5b-d324-4a8b-9c68-ea818b226ae4"
/>
This commit is contained in:
Bruno Quaresma
2025-03-19 15:19:57 -03:00
committed by GitHub
parent abbd88b819
commit ab763ca242
2 changed files with 26 additions and 25 deletions

View File

@ -67,9 +67,8 @@ export const MobileMenu: FC<MobileMenuProps> = ({
<DropdownMenuTrigger asChild>
<Button
aria-label={open ? "Close menu" : "Open menu"}
size="lg"
size="icon-lg"
variant="subtle"
className="ml-auto md:hidden"
>
{open ? <XIcon /> : <MenuIcon />}
</Button>

View File

@ -55,7 +55,7 @@ export const NavbarView: FC<NavbarViewProps> = ({
<NavItems className="ml-4" />
<div className=" hidden md:flex items-center gap-3 ml-auto">
<div className="hidden md:flex items-center gap-3 ml-auto">
{proxyContextValue && (
<ProxyMenu proxyContextValue={proxyContextValue} />
)}
@ -67,18 +67,6 @@ export const NavbarView: FC<NavbarViewProps> = ({
canViewHealth={canViewHealth}
/>
<NotificationsInbox
fetchNotifications={API.getInboxNotifications}
markAllAsRead={() => {
throw new Error("Function not implemented.");
}}
markNotificationAsRead={(notificationId) =>
API.updateInboxNotificationReadStatus(notificationId, {
is_read: true,
})
}
/>
{user && (
<UserDropdown
user={user}
@ -89,6 +77,19 @@ export const NavbarView: FC<NavbarViewProps> = ({
)}
</div>
<div className="ml-auto flex items-center gap-3 md:hidden">
<NotificationsInbox
fetchNotifications={API.getInboxNotifications}
markAllAsRead={() => {
throw new Error("Function not implemented.");
}}
markNotificationAsRead={(notificationId) =>
API.updateInboxNotificationReadStatus(notificationId, {
is_read: true,
})
}
/>
<MobileMenu
proxyContextValue={proxyContextValue}
user={user}
@ -100,6 +101,7 @@ export const NavbarView: FC<NavbarViewProps> = ({
canViewHealth={canViewHealth}
/>
</div>
</div>
);
};