From 8e330868f611ff10582117f844ff2f4eda9188ab Mon Sep 17 00:00:00 2001
From: Oliver <oliver.henry.walters@gmail.com>
Date: Thu, 3 Jul 2025 13:33:05 +1000
Subject: [PATCH] [UI] Notification Drawer (#9940)

- Handle error case
- Visual updates
---
 .../src/components/nav/NotificationDrawer.tsx | 29 +++++++++++++++----
 1 file changed, 23 insertions(+), 6 deletions(-)

diff --git a/src/frontend/src/components/nav/NotificationDrawer.tsx b/src/frontend/src/components/nav/NotificationDrawer.tsx
index ee5901c428..b8f93ea8ed 100644
--- a/src/frontend/src/components/nav/NotificationDrawer.tsx
+++ b/src/frontend/src/components/nav/NotificationDrawer.tsx
@@ -13,7 +13,12 @@ import {
   Text,
   Tooltip
 } from '@mantine/core';
-import { IconArrowRight, IconBellCheck } from '@tabler/icons-react';
+import {
+  IconArrowRight,
+  IconBellCheck,
+  IconCircleCheck,
+  IconExclamationCircle
+} from '@tabler/icons-react';
 import { useQuery } from '@tanstack/react-query';
 import { useCallback, useMemo } from 'react';
 import { useNavigate } from 'react-router-dom';
@@ -203,11 +208,14 @@ export function NotificationDrawer({
       <Boundary label='NotificationDrawer'>
         <Stack gap='xs'>
           <Divider />
-          {!hasNotifications && (
-            <Alert color='green'>
-              <Text size='sm'>{t`You have no unread notifications.`}</Text>
-            </Alert>
-          )}
+          {!notificationQuery.isFetching &&
+            !notificationQuery.isLoading &&
+            !notificationQuery.isError &&
+            !hasNotifications && (
+              <Alert color='green' icon={<IconCircleCheck />}>
+                <Text size='sm'>{t`You have no unread notifications.`}</Text>
+              </Alert>
+            )}
           {hasNotifications &&
             notificationQuery.data?.results?.map((notification: any) => (
               <NotificationEntry
@@ -221,6 +229,15 @@ export function NotificationDrawer({
               <Loader size='sm' />
             </Center>
           )}
+          {notificationQuery.isError && (
+            <Alert
+              color='red'
+              title={t`Error`}
+              icon={<IconExclamationCircle />}
+            >
+              <Text size='sm'>{t`Error loading notifications.`}</Text>
+            </Alert>
+          )}
         </Stack>
       </Boundary>
     </Drawer>