updated client review page2
Deploy / deploy (push) Successful in 2m32s

This commit is contained in:
twotalesanimation
2026-06-22 08:03:00 +02:00
parent ab062a589a
commit 1a5a56cf4c
+39 -39
View File
@@ -268,9 +268,9 @@ export default function ClientReviewPage({
return ( return (
<div className="flex flex-col h-screen bg-zinc-950 text-white overflow-hidden"> <div className="flex flex-col h-screen bg-zinc-950 text-white overflow-hidden">
{/* Header */} {/* Header */}
<header className="flex flex-col sm:flex-row sm:items-center gap-2 px-4 py-2.5 border-b border-zinc-800 bg-zinc-900 shrink-0"> <header className="grid grid-cols-[1fr_auto_1fr] items-center gap-2 px-4 py-2.5 border-b border-zinc-800 bg-zinc-900 shrink-0">
{/* Row 1: back + shot info + status */} {/* Left: back + shot path */}
<div className="flex items-center gap-3 flex-1 min-w-0"> <div className="flex items-center gap-3 min-w-0">
<Link <Link
href={`/client/${token}`} href={`/client/${token}`}
className="text-zinc-400 hover:text-white transition-colors flex items-center gap-1.5 text-sm shrink-0" className="text-zinc-400 hover:text-white transition-colors flex items-center gap-1.5 text-sm shrink-0"
@@ -281,7 +281,7 @@ export default function ClientReviewPage({
<div className="h-4 w-px bg-zinc-700 shrink-0" /> <div className="h-4 w-px bg-zinc-700 shrink-0" />
<div className="flex items-center gap-2 flex-1 min-w-0"> <div className="flex items-center gap-2 min-w-0">
<div className="w-5 h-5 rounded bg-black flex items-center justify-center shrink-0"> <div className="w-5 h-5 rounded bg-black flex items-center justify-center shrink-0">
<Image src="/logo.svg" alt="Logo" width={14} height={14} /> <Image src="/logo.svg" alt="Logo" width={14} height={14} />
</div> </div>
@@ -298,9 +298,7 @@ export default function ClientReviewPage({
<span className="text-xs text-zinc-500 hidden sm:block">{project?.code}</span> <span className="text-xs text-zinc-500 hidden sm:block">{project?.code}</span>
<span className="text-zinc-600">/</span> <span className="text-zinc-600">/</span>
{contextCode && ( {contextCode && (
<> <span className="font-mono font-semibold">{contextCode}</span>
<span className="font-mono font-semibold">{contextCode}</span>
</>
)} )}
{taskName && ( {taskName && (
<> <>
@@ -328,7 +326,10 @@ export default function ClientReviewPage({
); );
})()} })()}
</div> </div>
</div>
{/* Center: status badge + decision buttons */}
<div className="flex items-center gap-2 shrink-0">
<span <span
className={cn( className={cn(
"text-xs px-2.5 py-1 rounded-full border hidden sm:inline-flex items-center gap-1 shrink-0", "text-xs px-2.5 py-1 rounded-full border hidden sm:inline-flex items-center gap-1 shrink-0",
@@ -346,52 +347,51 @@ export default function ClientReviewPage({
)} )}
{currentApprovalStatus.replace(/_/g, " ")} {currentApprovalStatus.replace(/_/g, " ")}
</span> </span>
</div>
{/* Row 2 on mobile / inline on desktop: decision buttons */}
<div className="flex items-center gap-2 shrink-0">
{prevReview && (
<>
<Link
href={`/client/${token}/review/${prevReview.versionId}`}
className="inline-flex items-center gap-1 text-sm font-medium text-zinc-300 hover:text-white transition-colors shrink-0"
>
<ChevronLeft className="h-4 w-4" />
Prev
<span className="font-mono text-xs text-zinc-500 hidden sm:inline">{prevReview.label}</span>
</Link>
<div className="h-6 w-px bg-zinc-700" />
</>
)}
<Button <Button
size="sm" size="sm"
variant="outline" variant="outline"
className="flex-1 sm:flex-none h-9 sm:h-8 text-xs gap-1 text-orange-400 border-orange-500/30 hover:bg-orange-500/10" className="h-8 text-xs gap-1 text-orange-400 border-orange-500/30 hover:bg-orange-500/10"
onClick={() => setApprovalDialog({ open: true, status: "NEEDS_CHANGES" })} onClick={() => setApprovalDialog({ open: true, status: "NEEDS_CHANGES" })}
> >
<AlertCircle className="h-3.5 w-3.5" /> <AlertCircle className="h-3.5 w-3.5" />
Needs Changes <span className="hidden sm:inline">Needs Changes</span>
</Button> </Button>
<Button <Button
size="sm" size="sm"
className="flex-1 sm:flex-none h-9 sm:h-8 text-xs gap-1 bg-emerald-600 hover:bg-emerald-500 text-white" className="h-8 text-xs gap-1 bg-emerald-600 hover:bg-emerald-500 text-white"
onClick={() => setApprovalDialog({ open: true, status: "APPROVED" })} onClick={() => setApprovalDialog({ open: true, status: "APPROVED" })}
> >
<CheckCircle2 className="h-3.5 w-3.5" /> <CheckCircle2 className="h-3.5 w-3.5" />
Approve Approve
</Button> </Button>
{nextReview && ( </div>
<>
<div className="h-6 w-px bg-zinc-700" /> {/* Right: Prev / Next navigation */}
<Link <div className="flex items-center gap-2 justify-end shrink-0">
href={`/client/${token}/review/${nextReview.versionId}`} {prevReview ? (
className="inline-flex items-center gap-1 text-sm font-medium text-zinc-300 hover:text-white transition-colors shrink-0" <Link
> href={`/client/${token}/review/${prevReview.versionId}`}
<span className="font-mono text-xs text-zinc-500 hidden sm:inline">{nextReview.label}</span> className="inline-flex items-center gap-1 text-sm font-medium text-zinc-300 hover:text-white transition-colors shrink-0"
Next >
<ChevronRight className="h-4 w-4" /> <ChevronLeft className="h-4 w-4" />
</Link> <span className="hidden sm:inline">Prev</span>
</> <span className="font-mono text-xs text-zinc-500 hidden md:inline">{prevReview.label}</span>
</Link>
) : (
<span className="w-14" />
)}
{(prevReview || nextReview) && <div className="h-6 w-px bg-zinc-700" />}
{nextReview ? (
<Link
href={`/client/${token}/review/${nextReview.versionId}`}
className="inline-flex items-center gap-1 text-sm font-medium text-zinc-300 hover:text-white transition-colors shrink-0"
>
<span className="font-mono text-xs text-zinc-500 hidden md:inline">{nextReview.label}</span>
<span className="hidden sm:inline">Next</span>
<ChevronRight className="h-4 w-4" />
</Link>
) : (
<span className="w-14" />
)} )}
</div> </div>
</header> </header>