Francy51/Neon-Desk
2
0
Fork 0
Code Issues 22 Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: remove notification drawer and keep popover + detail modal

Browse Source
This commit is contained in:
francy51
2026-03-03 23:09:24 -05:00
parent da2ce23bab
commit 717e747869
1 changed files with 4 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
components/notifications/task-notifications-trigger.tsx
View File

@@ -21,7 +21,6 @@ type TaskNotificationsTriggerProps = {
silenceTask: (taskId: string, silenced?: boolean) => Promise<void>; silenceTask: (taskId: string, silenced?: boolean) => Promise<void>;
markTaskRead: (taskId: string, read?: boolean) => Promise<void>; markTaskRead: (taskId: string, read?: boolean) => Promise<void>;
className?: string; className?: string;
mobile?: boolean;
}; };
export function TaskNotificationsTrigger({ export function TaskNotificationsTrigger({
@@ -37,8 +36,7 @@ export function TaskNotificationsTrigger({
openTaskDetails, openTaskDetails,
silenceTask, silenceTask,
markTaskRead, markTaskRead,
className, className
mobile = false
}: TaskNotificationsTriggerProps) { }: TaskNotificationsTriggerProps) {
const button = ( const button = (
<button <button
@@ -46,18 +44,13 @@ export function TaskNotificationsTrigger({
aria-label="Open notifications" aria-label="Open notifications"
onClick={() => setIsPopoverOpen(!isPopoverOpen)} onClick={() => setIsPopoverOpen(!isPopoverOpen)}
className={cn( className={cn(
'relative inline-flex items-center justify-center rounded-lg border border-[color:var(--line-weak)] bg-[color:var(--panel-soft)] text-[color:var(--terminal-bright)] transition hover:border-[color:var(--line-strong)] hover:bg-[color:var(--panel-bright)]', 'relative inline-flex h-10 w-10 items-center justify-center rounded-lg border border-[color:var(--line-weak)] bg-[color:var(--panel-soft)] text-[color:var(--terminal-bright)] transition hover:border-[color:var(--line-strong)] hover:bg-[color:var(--panel-bright)]',
mobile ? 'min-w-0 flex-1 gap-1 px-2 py-1.5 text-[11px]' : 'h-10 w-10',
className className
)} )}
> >
{unreadCount > 0 ? <BellRing className="size-4" /> : <Bell className="size-4" />} {unreadCount > 0 ? <BellRing className="size-4" /> : <Bell className="size-4" />}
{mobile ? <span>Alerts</span> : null}
{unreadCount > 0 ? ( {unreadCount > 0 ? (
<span className={cn( <span className="absolute -right-1.5 -top-1.5 inline-flex min-w-[1.15rem] items-center justify-center rounded-full bg-[color:var(--accent)] px-1 text-[10px] font-semibold text-[#00241d]">
'absolute inline-flex min-w-[1.15rem] items-center justify-center rounded-full bg-[color:var(--accent)] px-1 text-[10px] font-semibold text-[#00241d]',
mobile ? 'right-1 top-1' : '-right-1.5 -top-1.5'
)}>
{unreadCount > 99 ? '99+' : unreadCount} {unreadCount > 99 ? '99+' : unreadCount}
</span> </span>
) : null} ) : null}
@@ -77,10 +70,7 @@ export function TaskNotificationsTrigger({
/> />
<div <div
className={cn( className={cn(
'z-50 rounded-xl border border-[color:var(--line-weak)] bg-[color:var(--panel)] p-3 shadow-[0_18px_50px_rgba(0,0,0,0.45)]', 'absolute right-0 z-50 mt-2 h-[34rem] max-h-[calc(100vh-8rem)] w-[min(27rem,calc(100vw-2rem))] rounded-xl border border-[color:var(--line-weak)] bg-[color:var(--panel)] p-3 shadow-[0_18px_50px_rgba(0,0,0,0.45)]'
mobile
? 'fixed inset-x-3 bottom-20 top-16 overflow-hidden'
: 'absolute right-0 mt-2 h-[34rem] w-[27rem]'
)} )}
> >
<div className="mb-2 flex items-center justify-between"> <div className="mb-2 flex items-center justify-between">