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.
This commit is contained in:
@@ -287,13 +287,11 @@ export function ChannelList() {
|
|||||||
{hasUnread(channel.id) && (
|
{hasUnread(channel.id) && (
|
||||||
<span className="text-gb-orange text-xs font-bold" title="Unread messages">●</span>
|
<span className="text-gb-orange text-xs font-bold" title="Unread messages">●</span>
|
||||||
)}
|
)}
|
||||||
<span className="flex items-center gap-1 text-xs">
|
<span className="flex items-center gap-1 text-xs pointer-events-none group-hover:pointer-events-auto">
|
||||||
<span
|
<span
|
||||||
onClick={(e) => handleNotifClick(e, channel.id)}
|
onClick={(e) => handleNotifClick(e, channel.id)}
|
||||||
className="text-gb-fg-f hover:text-gb-orange opacity-0 group-hover:opacity-100 transition-opacity cursor-pointer"
|
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)]}`}
|
title={`Notifications: ${NOTIF_LABELS[getLevel(channel.id)]}`}
|
||||||
role="button"
|
|
||||||
tabIndex={-1}
|
|
||||||
>
|
>
|
||||||
{notifIcon(channel.id)}
|
{notifIcon(channel.id)}
|
||||||
</span>
|
</span>
|
||||||
@@ -301,8 +299,6 @@ export function ChannelList() {
|
|||||||
onClick={(e) => { e.stopPropagation(); setSettingsChannel({ id: channel.id, name: channel.name }); }}
|
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"
|
className="text-gb-fg-f hover:text-gb-orange opacity-0 group-hover:opacity-100 cursor-pointer"
|
||||||
title="Channel settings"
|
title="Channel settings"
|
||||||
role="button"
|
|
||||||
tabIndex={-1}
|
|
||||||
>
|
>
|
||||||
[⚙]
|
[⚙]
|
||||||
</span>
|
</span>
|
||||||
|
|||||||
Reference in New Issue
Block a user