From 537f37c33e325c577f5291bc9438bb52dd54ecf3 Mon Sep 17 00:00:00 2001
From: jdjioe5-cpu <jdjioe5-cpu@users.noreply.github.com>
Date: Sun, 26 Jul 2026 12:44:04 +0800
Subject: [PATCH] fix(notification-bell): add data-testid to trigger button
 (issue #380)

Closes FlowwStar/FlowStar#380.

Adds data-testid="notification-bell-trigger" to the bell trigger button
so e2e tests can target the bell independently of the dynamic
aria-label (which varies with unread count).

No behavior change.
---
 components/layout/notification-bell.tsx | 1 +
 1 file changed, 1 insertion(+)

diff --git a/components/layout/notification-bell.tsx b/components/layout/notification-bell.tsx
index 62b2e0e..78edda0 100644
--- a/components/layout/notification-bell.tsx
+++ b/components/layout/notification-bell.tsx
@@ -79,6 +79,7 @@ export function NotificationBell() {
         onClick={handleToggle}
         aria-expanded={open}
         aria-haspopup="true"
+        data-testid="notification-bell-trigger"
         className="relative rounded-md p-2 text-muted-foreground transition-colors hover:text-foreground focus-visible:outline-none focus-visible:ring-2 focus-visible:ring-ring"
         aria-label={`Notifications${unreadCount > 0 ? ` (${unreadCount} unread)` : ''}`}
       >