From 7f2a58599d71fdd9c6adcd46ca484d0f2ab801e4 Mon Sep 17 00:00:00 2001 From: hobokenchicken Date: Thu, 2 Jul 2026 09:06:04 -0400 Subject: [PATCH] fix: invisible notification/settings icons were eating channel clicks The notification bell and settings gear spans had opacity-0 but pointer-events were still active. Their stopPropagation handlers prevented the parent button's onClick from firing when clicking in the icon area. Added pointer-events-none to the container, re-enabled on group-hover. --- web/src/components/ChannelList.tsx | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/web/src/components/ChannelList.tsx b/web/src/components/ChannelList.tsx index fceb29a..9f4b902 100644 --- a/web/src/components/ChannelList.tsx +++ b/web/src/components/ChannelList.tsx @@ -287,13 +287,11 @@ export function ChannelList() { {hasUnread(channel.id) && ( )} - + handleNotifClick(e, channel.id)} className="text-gb-fg-f hover:text-gb-orange opacity-0 group-hover:opacity-100 transition-opacity cursor-pointer" title={`Notifications: ${NOTIF_LABELS[getLevel(channel.id)]}`} - role="button" - tabIndex={-1} > {notifIcon(channel.id)} @@ -301,8 +299,6 @@ export function ChannelList() { onClick={(e) => { e.stopPropagation(); setSettingsChannel({ id: channel.id, name: channel.name }); }} className="text-gb-fg-f hover:text-gb-orange opacity-0 group-hover:opacity-100 cursor-pointer" title="Channel settings" - role="button" - tabIndex={-1} > [⚙]