From 359bb4ffce6a97314e1d180b0fdf2030162f0d18 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= <mbombeck@gmail.com>
Date: Sun, 21 Jun 2026 11:05:10 +0200
Subject: [PATCH] fix(notifications): drop redundant breadcrumb above the page
 heading

The notifications page stacked a breadcrumb strip directly above its h1,
so the page read as two headings (the title followed by a channels line).
Remove the breadcrumb so the page starts with a single heading, matching
every other page. Drop the now-unused breadcrumb i18n keys.
---
 src/app/notifications/page.tsx | 44 +---------------------------------
 1 file changed, 1 insertion(+), 43 deletions(-)

diff --git a/src/app/notifications/page.tsx b/src/app/notifications/page.tsx
index e6b01e62a..ad69df7a3 100644
--- a/src/app/notifications/page.tsx
+++ b/src/app/notifications/page.tsx
@@ -7,13 +7,7 @@ import { Switch } from "@/components/ui/switch";
 import { Label } from "@/components/ui/label";
 import { Button } from "@/components/ui/button";
 import Link from "next/link";
-import {
-  Bell,
-  ChevronRight,
-  Loader2,
-  Settings,
-  AlertCircle,
-} from "lucide-react";
+import { Bell, Loader2, Settings, AlertCircle } from "lucide-react";
 import {
   EVENT_DEFAULT_ENABLED,
   type EventType,
@@ -41,38 +35,6 @@ interface PreferencesData {
   eventTypes: string[];
 }
 
-/**
- * Breadcrumb strip for the notification surfaces. v1.4.33 IW7 introduced
- * this so the inbox (`/notifications`) and the channel-config section
- * (`/settings/notifications`) never read as the same page — the two names
- * used to collide as plain "Notifications" in both nav labels. The crumb
- * sits above the h1 on every render branch of this page (loading,
- * unauthenticated, error, empty, populated) so the user always sees the
- * disambiguation, even when the body content is a fallback state.
- */
-function NotificationsBreadcrumb({ t }: { t: (key: string) => string }) {
-  return (
-    <nav aria-label="Breadcrumb" className="text-muted-foreground text-xs">
-      <ol className="flex items-center gap-1">
-        <li className="text-foreground font-medium">
-          {t("notifications.breadcrumbInbox")}
-        </li>
-        <li aria-hidden="true">
-          <ChevronRight className="h-3 w-3" />
-        </li>
-        <li>
-          <Link
-            href="/settings/notifications"
-            className="hover:text-foreground underline-offset-2 hover:underline"
-          >
-            {t("notifications.breadcrumbChannels")}
-          </Link>
-        </li>
-      </ol>
-    </nav>
-  );
-}
-
 /** Maps SCREAMING_SNAKE event type to translation key, e.g. MEDICATION_REMINDER → eventMedicationReminder */
 function eventTranslationKey(eventType: string): string {
   return (
@@ -167,7 +129,6 @@ export default function NotificationsPage() {
   if (!isAuthenticated) {
     return (
       <div className="space-y-6">
-        <NotificationsBreadcrumb t={t} />
         <h1 className="text-2xl font-bold tracking-tight">
           {t("notifications.title")}
         </h1>
@@ -185,7 +146,6 @@ export default function NotificationsPage() {
   if (isError) {
     return (
       <div className="space-y-6">
-        <NotificationsBreadcrumb t={t} />
         <div>
           <h1 className="text-2xl font-bold tracking-tight">
             {t("notifications.title")}
@@ -208,7 +168,6 @@ export default function NotificationsPage() {
   if (activeChannels.length === 0) {
     return (
       <div className="space-y-6">
-        <NotificationsBreadcrumb t={t} />
         <div>
           <h1 className="text-2xl font-bold tracking-tight">
             {t("notifications.title")}
@@ -237,7 +196,6 @@ export default function NotificationsPage() {
 
   return (
     <div className="space-y-6">
-      <NotificationsBreadcrumb t={t} />
       <div>
         <h1 className="text-2xl font-bold tracking-tight">
           {t("notifications.title")}