feat: three-column ticket list layout (list + detail as peers, no Sheet)

- Replace Sheet slide-over with persistent right-column detail panel
- Ticket list shrinks to w-80 when ticket selected, detail takes flex-1
- Animated transition (300ms ease-out) when selecting/deselecting
- Kept existing conversation thread, properties sidebar, reply box inline
This commit is contained in:
Gjermund Høsøien Wiggen
2026-06-07 22:58:50 +02:00
parent 784d30acbd
commit 10962f795f

View File

@@ -16,12 +16,6 @@ import {
DialogDescription, DialogDescription,
DialogFooter, DialogFooter,
} from "@/components/ui/dialog"; } from "@/components/ui/dialog";
import {
Sheet,
SheetContent,
SheetHeader,
SheetTitle,
} from "@/components/ui/sheet";
import { Textarea } from "@/components/ui/textarea"; import { Textarea } from "@/components/ui/textarea";
import { cn } from "@/lib/utils"; import { cn } from "@/lib/utils";
@@ -66,7 +60,7 @@ function getInitialColor(name: string): string {
return colors[Math.abs(hash) % colors.length]; return colors[Math.abs(hash) % colors.length];
} }
function TicketRow({ ticket, onClick }: { ticket: Ticket; onClick: () => void }) { function TicketRow({ ticket, selected, onClick }: { ticket: Ticket; selected: boolean; onClick: () => void }) {
const statusColor = STATUS_COLORS[ticket.status] || STATUS_COLORS.new; const statusColor = STATUS_COLORS[ticket.status] || STATUS_COLORS.new;
const shortId = ticket.id.slice(0, 8); const shortId = ticket.id.slice(0, 8);
const timeAgo = formatDistanceToNow(new Date(ticket.updated_at), { addSuffix: true }); const timeAgo = formatDistanceToNow(new Date(ticket.updated_at), { addSuffix: true });
@@ -74,7 +68,12 @@ function TicketRow({ ticket, onClick }: { ticket: Ticket; onClick: () => void })
return ( return (
<button <button
onClick={onClick} onClick={onClick}
className="w-full flex items-center gap-3 py-3 px-4 text-left hover:bg-accent transition-all duration-150 border-l-2 border-transparent hover:border-primary border-b border-border" className={cn(
"w-full flex items-center gap-3 py-3 px-4 text-left transition-all duration-150 border-l-2 border-b border-border",
selected
? "bg-accent border-l-primary"
: "border-l-transparent hover:bg-accent hover:border-l-primary"
)}
> >
<span <span
className="w-2 h-2 rounded-full flex-shrink-0" className="w-2 h-2 rounded-full flex-shrink-0"
@@ -120,236 +119,240 @@ function SkeletonRow() {
); );
} }
function TicketDetailSheet({ function TicketDetailPanel({
ticketId, ticketId,
open, onBack,
onOpenChange,
}: { }: {
ticketId: string | null; ticketId: string;
open: boolean; onBack: () => void;
onOpenChange: (open: boolean) => void;
}) { }) {
const [ticket, setTicket] = useState<Ticket | null>(null); const [ticket, setTicket] = useState<Ticket | null>(null);
const [transactions, setTransactions] = useState<Transaction[]>([]); const [transactions, setTransactions] = useState<Transaction[]>([]);
const [loading, setLoading] = useState(false); const [loading, setLoading] = useState(true);
const [error, setError] = useState<string | null>(null); const [error, setError] = useState<string | null>(null);
const [replyText, setReplyText] = useState(""); const [replyText, setReplyText] = useState("");
useEffect(() => { useEffect(() => {
if (open && ticketId) { setLoading(true);
setLoading(true); setError(null);
setError(null); setTicket(null);
Promise.all([ setTransactions([]);
getTicket(ticketId), setReplyText("");
getTicketTransactions(ticketId), Promise.all([
]).then(([ticketRes, txRes]) => { getTicket(ticketId),
if (ticketRes.error) { getTicketTransactions(ticketId),
setError(ticketRes.error); ]).then(([ticketRes, txRes]) => {
} else { if (ticketRes.error) {
setTicket(ticketRes.data); setError(ticketRes.error);
} } else {
if (txRes.data) { setTicket(ticketRes.data);
setTransactions(txRes.data); }
} if (txRes.data) {
setLoading(false); setTransactions(txRes.data);
}); }
} setLoading(false);
}, [open, ticketId]); });
}, [ticketId]);
return ( return (
<Sheet open={open} onOpenChange={onOpenChange}> <div className="flex flex-col h-full">
<SheetContent {/* Header */}
side="right" <div className="flex items-center gap-3 px-4 py-2.5 border-b border-border shrink-0">
className="w-full sm:max-w-lg flex flex-col p-0" <button
showCloseButton={true} onClick={onBack}
> className="flex items-center gap-1.5 text-xs text-muted-foreground hover:text-foreground transition-all duration-150 flex-shrink-0"
{loading && ( >
<div className="flex-1 p-4 space-y-4"> <ArrowLeftIcon className="w-3.5 h-3.5" />
{Array.from({ length: 6 }).map((_, i) => ( Back
<div key={i} className="flex gap-3"> </button>
<div className="w-6 h-6 rounded-full bg-muted animate-pulse" /> </div>
<div className="flex-1 space-y-2">
<div className="h-3 bg-muted rounded animate-pulse w-3/4" /> {loading && (
<div className="h-3 bg-muted rounded animate-pulse w-1/2" /> <div className="flex-1 p-4 space-y-4 overflow-y-auto">
</div> {Array.from({ length: 6 }).map((_, i) => (
<div key={i} className="flex gap-3">
<div className="w-6 h-6 rounded-full bg-muted animate-pulse" />
<div className="flex-1 space-y-2">
<div className="h-3 bg-muted rounded animate-pulse w-3/4" />
<div className="h-3 bg-muted rounded animate-pulse w-1/2" />
</div> </div>
))} </div>
))}
</div>
)}
{error && (
<div className="flex flex-col items-center justify-center flex-1">
<p className="text-sm text-destructive">{error}</p>
</div>
)}
{!loading && !error && ticket && (
<>
{/* Title */}
<div className="px-4 py-3 border-b border-border shrink-0">
<h2 className="text-sm font-semibold truncate">
{ticket.subject}
</h2>
<p className="text-xs text-muted-foreground mt-0.5">
<span className="font-mono">{ticket.id.slice(0, 8)}</span>
</p>
</div> </div>
)}
{error && ( {/* Conversation */}
<div className="flex flex-col items-center justify-center flex-1"> <div className="flex-1 overflow-y-auto">
<p className="text-sm text-destructive">{error}</p> {transactions.length === 0 && (
</div> <div className="flex flex-col items-center justify-center py-20">
)} <p className="text-sm text-muted-foreground">
No activity yet
</p>
</div>
)}
{transactions.map((tx) => {
const isSystem =
tx.transaction_type === "status_change" ||
tx.transaction_type === "assignment" ||
tx.transaction_type === "create";
const isAgent =
tx.transaction_type === "agent_reply" ||
tx.transaction_type === "internal_note";
{!loading && !error && ticket && ( if (isSystem) {
<div className="flex flex-col h-full">
{/* Header */}
<SheetHeader className="border-b border-border shrink-0">
<SheetTitle className="text-sm font-semibold truncate">
{ticket.subject}
</SheetTitle>
<p className="text-xs text-muted-foreground">
<span className="font-mono">{ticket.id.slice(0, 8)}</span>
</p>
</SheetHeader>
{/* Conversation */}
<div className="flex-1 overflow-y-auto">
{transactions.length === 0 && (
<div className="flex flex-col items-center justify-center py-20">
<p className="text-sm text-muted-foreground">
No activity yet
</p>
</div>
)}
{transactions.map((tx) => {
const isSystem =
tx.transaction_type === "status_change" ||
tx.transaction_type === "assignment" ||
tx.transaction_type === "create";
const isAgent =
tx.transaction_type === "agent_reply" ||
tx.transaction_type === "internal_note";
if (isSystem) {
const txTimeAgo = formatDistanceToNow(
new Date(tx.created_at),
{ addSuffix: true }
);
let message = "";
if (tx.transaction_type === "create") {
message = "Ticket created";
} else if (tx.transaction_type === "status_change") {
const oldLabel = tx.old_value
? STATUS_LABELS[tx.old_value] || tx.old_value
: "?";
const newLabel = tx.new_value
? STATUS_LABELS[tx.new_value] || tx.new_value
: "?";
message = `${getInitial(tx.creator_id)} changed status from ${oldLabel} to ${newLabel}`;
} else if (tx.transaction_type === "assignment") {
message = tx.new_value
? `${getInitial(tx.creator_id)} assigned to ${tx.new_value}`
: `${getInitial(tx.creator_id)} unassigned`;
} else {
message = tx.transaction_type;
}
return (
<div
key={tx.id}
className="flex justify-center py-2"
>
<span className="text-xs text-muted-foreground">
{message} · {txTimeAgo}
</span>
</div>
);
}
const isInternal =
tx.transaction_type === "internal_note";
const txTimeAgo = formatDistanceToNow( const txTimeAgo = formatDistanceToNow(
new Date(tx.created_at), new Date(tx.created_at),
{ addSuffix: true } { addSuffix: true }
); );
let message = "";
if (tx.transaction_type === "create") {
message = "Ticket created";
} else if (tx.transaction_type === "status_change") {
const oldLabel = tx.old_value
? STATUS_LABELS[tx.old_value] || tx.old_value
: "?";
const newLabel = tx.new_value
? STATUS_LABELS[tx.new_value] || tx.new_value
: "?";
message = `${getInitial(tx.creator_id)} changed status from ${oldLabel} to ${newLabel}`;
} else if (tx.transaction_type === "assignment") {
message = tx.new_value
? `${getInitial(tx.creator_id)} assigned to ${tx.new_value}`
: `${getInitial(tx.creator_id)} unassigned`;
} else {
message = tx.transaction_type;
}
return ( return (
<div <div
key={tx.id} key={tx.id}
className={cn( className="flex justify-center py-2"
"flex gap-3 py-3 px-4",
isAgent ? "flex-row-reverse" : ""
)}
> >
<div <span className="text-xs text-muted-foreground">
className="w-6 h-6 rounded-full flex items-center justify-center flex-shrink-0" {message} · {txTimeAgo}
style={{ </span>
backgroundColor: isAgent
? getInitialColor(tx.creator_id)
: "var(--muted)",
}}
>
<span
className="text-[11px] font-semibold"
style={{
color: isAgent
? "#f7f8f8"
: "var(--muted-foreground)",
}}
>
{getInitial(tx.creator_id)}
</span>
</div>
<div
className={cn(
"max-w-[75%] rounded-lg px-3 py-2",
isAgent
? "bg-primary/15 text-foreground"
: isInternal
? "bg-muted border border-border text-foreground"
: "bg-muted text-foreground"
)}
>
{isInternal && (
<div className="text-[10px] font-semibold text-chart-3 mb-0.5 uppercase tracking-wider">
Internal note
</div>
)}
<p className="text-sm whitespace-pre-wrap">
{typeof tx.data === "object" &&
tx.data !== null &&
"body" in (tx.data as Record<string, unknown>)
? String(
(tx.data as Record<string, unknown>).body
)
: tx.transaction_type}
</p>
<p className="text-[10px] text-muted-foreground mt-1">
{txTimeAgo}
</p>
</div>
</div> </div>
); );
})} }
</div>
{/* Reply box */} const isInternal =
<div className="border-t border-border bg-sidebar p-3 shrink-0"> tx.transaction_type === "internal_note";
<div className="flex items-end gap-2"> const txTimeAgo = formatDistanceToNow(
<textarea new Date(tx.created_at),
value={replyText} { addSuffix: true }
onChange={(e) => setReplyText(e.target.value)} );
placeholder="Reply to this ticket..."
rows={2} return (
className="flex-1 px-3 py-2 rounded-lg bg-background border border-border text-sm text-foreground placeholder:text-muted-foreground outline-none focus:border-primary focus:ring-1 focus:ring-primary resize-none transition-all duration-150" <div
/> key={tx.id}
<div className="flex items-center gap-1"> className={cn(
<button "flex gap-3 py-3 px-4",
className="w-8 h-8 flex items-center justify-center rounded-lg text-muted-foreground hover:text-foreground hover:bg-accent transition-all duration-150" isAgent ? "flex-row-reverse" : ""
title="Attach file (coming soon)" )}
>
<div
className="w-6 h-6 rounded-full flex items-center justify-center flex-shrink-0"
style={{
backgroundColor: isAgent
? getInitialColor(tx.creator_id)
: "var(--muted)",
}}
> >
<PaperclipIcon className="w-4 h-4" /> <span
</button> className="text-[11px] font-semibold"
<button style={{
disabled={!replyText.trim()} color: isAgent
? "#f7f8f8"
: "var(--muted-foreground)",
}}
>
{getInitial(tx.creator_id)}
</span>
</div>
<div
className={cn( className={cn(
"w-8 h-8 flex items-center justify-center rounded-lg transition-all duration-150", "max-w-[75%] rounded-lg px-3 py-2",
replyText.trim() isAgent
? "bg-primary text-primary-foreground hover:bg-primary/80" ? "bg-primary/15 text-foreground"
: "bg-muted text-muted-foreground cursor-not-allowed" : isInternal
? "bg-muted border border-border text-foreground"
: "bg-muted text-foreground"
)} )}
> >
<SendIcon className="w-4 h-4" /> {isInternal && (
</button> <div className="text-[10px] font-semibold text-chart-3 mb-0.5 uppercase tracking-wider">
Internal note
</div>
)}
<p className="text-sm whitespace-pre-wrap">
{typeof tx.data === "object" &&
tx.data !== null &&
"body" in (tx.data as Record<string, unknown>)
? String(
(tx.data as Record<string, unknown>).body
)
: tx.transaction_type}
</p>
<p className="text-[10px] text-muted-foreground mt-1">
{txTimeAgo}
</p>
</div>
</div> </div>
);
})}
</div>
{/* Reply box */}
<div className="border-t border-border bg-sidebar p-3 shrink-0">
<div className="flex items-end gap-2">
<textarea
value={replyText}
onChange={(e) => setReplyText(e.target.value)}
placeholder="Reply to this ticket..."
rows={2}
className="flex-1 px-3 py-2 rounded-lg bg-background border border-border text-sm text-foreground placeholder:text-muted-foreground outline-none focus:border-primary focus:ring-1 focus:ring-primary resize-none transition-all duration-150"
/>
<div className="flex items-center gap-1">
<button
className="w-8 h-8 flex items-center justify-center rounded-lg text-muted-foreground hover:text-foreground hover:bg-accent transition-all duration-150"
title="Attach file (coming soon)"
>
<PaperclipIcon className="w-4 h-4" />
</button>
<button
disabled={!replyText.trim()}
className={cn(
"w-8 h-8 flex items-center justify-center rounded-lg transition-all duration-150",
replyText.trim()
? "bg-primary text-primary-foreground hover:bg-primary/80"
: "bg-muted text-muted-foreground cursor-not-allowed"
)}
>
<SendIcon className="w-4 h-4" />
</button>
</div> </div>
</div> </div>
</div> </div>
)} </>
</SheetContent> )}
</Sheet> </div>
); );
} }
@@ -373,9 +376,8 @@ function TicketListPageContent() {
const [submitting, setSubmitting] = useState(false); const [submitting, setSubmitting] = useState(false);
const [createError, setCreateError] = useState<string | null>(null); const [createError, setCreateError] = useState<string | null>(null);
// Sheet // Selected ticket
const [sheetTicketId, setSheetTicketId] = useState<string | null>(null); const [selectedTicketId, setSelectedTicketId] = useState<string | null>(null);
const [sheetOpen, setSheetOpen] = useState(false);
const initialQueueId = searchParams.get("queue") || ""; const initialQueueId = searchParams.get("queue") || "";
@@ -456,106 +458,133 @@ function TicketListPageContent() {
setNewSubject(""); setNewSubject("");
setNewQueueId(""); setNewQueueId("");
setNewDescription(""); setNewDescription("");
// Open in sheet setSelectedTicketId(data.id);
setSheetTicketId(data.id); // Refresh the list to show the new ticket
setSheetOpen(true); fetchData();
} }
}; };
const handleSelectTicket = (ticketId: string) => {
setSelectedTicketId(ticketId);
};
const handleDeselectTicket = () => {
setSelectedTicketId(null);
};
return ( return (
<div className="flex flex-col h-full"> <div className="flex h-full">
{/* Top bar */} {/* Ticket list column */}
<div className="flex items-center gap-3 px-4 py-3 border-b border-border"> <div
<div className="relative flex-1"> className={cn(
<SearchIcon className="absolute left-3 top-1/2 -translate-y-1/2 w-4 h-4 text-muted-foreground" /> "flex flex-col h-full overflow-hidden transition-all duration-300 ease-out",
<input selectedTicketId ? "w-80 min-w-80 flex-shrink-0" : "flex-1 min-w-0"
value={searchQuery}
onChange={(e) => setSearchQuery(e.target.value)}
placeholder="Search tickets..."
className="w-full h-8 pl-9 pr-3 rounded-lg bg-background border border-border text-sm text-foreground placeholder:text-muted-foreground outline-none focus:border-primary focus:ring-1 focus:ring-primary transition-all duration-150"
/>
</div>
<Button
onClick={() => setDialogOpen(true)}
className="h-8 px-3 rounded-lg text-sm font-medium bg-gradient-to-r from-primary to-primary/80 hover:from-primary/90 hover:to-primary/70 text-primary-foreground border-0 transition-all duration-150"
>
<PlusIcon className="w-4 h-4 mr-1.5" />
New ticket
</Button>
</div>
{/* Filter chips */}
<div className="flex items-center gap-1.5 px-4 py-2 border-b border-border">
{FILTERS.map((filter) => (
<button
key={filter.key ?? "all"}
onClick={() => setActiveFilter(filter.key)}
className={cn(
"px-2.5 py-1 rounded-md text-xs font-medium transition-all duration-150",
activeFilter === filter.key
? "bg-primary text-primary-foreground"
: "text-muted-foreground hover:text-foreground hover:bg-accent"
)}
>
{filter.label}
</button>
))}
</div>
{/* Ticket list */}
<div className="flex-1 overflow-y-auto">
{error && (
<div className="px-4 py-3 text-sm text-destructive bg-destructive/5 border-b border-destructive/10">
{error}{" "}
<button
onClick={fetchData}
className="underline hover:text-destructive/80"
>
Retry
</button>
</div>
)} )}
>
{/* Top bar */}
<div className="flex items-center gap-3 px-4 py-3 border-b border-border shrink-0">
<div className="relative flex-1">
<SearchIcon className="absolute left-3 top-1/2 -translate-y-1/2 w-4 h-4 text-muted-foreground" />
<input
value={searchQuery}
onChange={(e) => setSearchQuery(e.target.value)}
placeholder="Search tickets..."
className="w-full h-8 pl-9 pr-3 rounded-lg bg-background border border-border text-sm text-foreground placeholder:text-muted-foreground outline-none focus:border-primary focus:ring-1 focus:ring-primary transition-all duration-150"
/>
</div>
<Button
onClick={() => setDialogOpen(true)}
className="h-8 px-3 rounded-lg text-sm font-medium bg-gradient-to-r from-primary to-primary/80 hover:from-primary/90 hover:to-primary/70 text-primary-foreground border-0 transition-all duration-150"
>
<PlusIcon className="w-4 h-4 mr-1.5" />
New ticket
</Button>
</div>
{loading && {/* Filter chips */}
Array.from({ length: 12 }).map((_, i) => <SkeletonRow key={i} />)} <div className="flex items-center gap-1.5 px-4 py-2 border-b border-border shrink-0">
{FILTERS.map((filter) => (
<button
key={filter.key ?? "all"}
onClick={() => setActiveFilter(filter.key)}
className={cn(
"px-2.5 py-1 rounded-md text-xs font-medium transition-all duration-150",
activeFilter === filter.key
? "bg-primary text-primary-foreground"
: "text-muted-foreground hover:text-foreground hover:bg-accent"
)}
>
{filter.label}
</button>
))}
</div>
{!loading && {/* Ticket list */}
!error && <div className="flex-1 overflow-y-auto">
tickets.length === 0 && ( {error && (
<div className="flex flex-col items-center justify-center py-20 px-4"> <div className="px-4 py-3 text-sm text-destructive bg-destructive/5 border-b border-destructive/10">
<div className="w-12 h-12 rounded-full bg-muted flex items-center justify-center mb-4"> {error}{" "}
<SearchIcon className="w-5 h-5 text-muted-foreground" /> <button
</div> onClick={fetchData}
<p className="text-sm text-foreground font-medium"> className="underline hover:text-destructive/80"
No tickets match your filters
</p>
<p className="text-xs text-muted-foreground mt-1">
Try adjusting your search or filter criteria
</p>
<Button
onClick={() => setDialogOpen(true)}
className="mt-4 h-8 px-3 rounded-lg text-sm font-medium bg-gradient-to-r from-primary to-primary/80 hover:from-primary/90 hover:to-primary/70 text-primary-foreground border-0 transition-all duration-150"
> >
<PlusIcon className="w-4 h-4 mr-1.5" /> Retry
Create a ticket </button>
</Button>
</div> </div>
)} )}
{!loading && {loading &&
!error && Array.from({ length: 12 }).map((_, i) => <SkeletonRow key={i} />)}
tickets.map((ticket) => (
<TicketRow {!loading &&
key={ticket.id} !error &&
ticket={ticket} tickets.length === 0 && (
onClick={() => { <div className="flex flex-col items-center justify-center py-20 px-4">
setSheetTicketId(ticket.id); <div className="w-12 h-12 rounded-full bg-muted flex items-center justify-center mb-4">
setSheetOpen(true); <SearchIcon className="w-5 h-5 text-muted-foreground" />
}} </div>
/> <p className="text-sm text-foreground font-medium">
))} No tickets match your filters
</p>
<p className="text-xs text-muted-foreground mt-1">
Try adjusting your search or filter criteria
</p>
<Button
onClick={() => setDialogOpen(true)}
className="mt-4 h-8 px-3 rounded-lg text-sm font-medium bg-gradient-to-r from-primary to-primary/80 hover:from-primary/90 hover:to-primary/70 text-primary-foreground border-0 transition-all duration-150"
>
<PlusIcon className="w-4 h-4 mr-1.5" />
Create a ticket
</Button>
</div>
)}
{!loading &&
!error &&
tickets.map((ticket) => (
<TicketRow
key={ticket.id}
ticket={ticket}
selected={ticket.id === selectedTicketId}
onClick={() => handleSelectTicket(ticket.id)}
/>
))}
</div>
</div> </div>
{/* Ticket detail column */}
{selectedTicketId && (
<div
className="flex-1 min-w-0 border-l overflow-hidden"
style={{ animation: "slide-in-right 300ms ease-out" }}
>
<TicketDetailPanel
ticketId={selectedTicketId}
onBack={handleDeselectTicket}
/>
</div>
)}
{/* Create ticket dialog */} {/* Create ticket dialog */}
<Dialog open={dialogOpen} onOpenChange={setDialogOpen}> <Dialog open={dialogOpen} onOpenChange={setDialogOpen}>
<DialogContent <DialogContent
@@ -654,13 +683,6 @@ function TicketListPageContent() {
</DialogFooter> </DialogFooter>
</DialogContent> </DialogContent>
</Dialog> </Dialog>
{/* Ticket detail sheet */}
<TicketDetailSheet
ticketId={sheetTicketId}
open={sheetOpen}
onOpenChange={setSheetOpen}
/>
</div> </div>
); );
} }