fix: Fix sidebar layout — sticky height constraint, padding on content only

This commit is contained in:
Lukas May
2026-03-05 11:12:06 +01:00
parent a9832fc5ae
commit 41cbc2c76c
2 changed files with 14 additions and 12 deletions

View File

@@ -37,7 +37,7 @@ export function ReviewSidebar({
return (
<div className="flex h-full">
{/* Content panel */}
<div className="flex-1 min-w-0 overflow-y-auto">
<div className="flex-1 min-w-0 overflow-y-auto p-4">
{view === "files" ? (
<FilesView
files={files}

View File

@@ -261,8 +261,9 @@ export function ReviewTab({ initiativeId }: ReviewTabProps) {
)}
</div>
{/* Right: Sidebar */}
<div className="border-l border-border p-4">
{/* Right: Sidebar — sticky so icon strip stays visible */}
<div className="border-l border-border">
<div className="sticky top-0 h-[calc(100vh-12rem)]">
<ReviewSidebar
files={allFiles}
comments={comments}
@@ -274,6 +275,7 @@ export function ReviewTab({ initiativeId }: ReviewTabProps) {
/>
</div>
</div>
</div>
)}
</div>
);