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 ( return (
<div className="flex h-full"> <div className="flex h-full">
{/* Content panel */} {/* 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" ? ( {view === "files" ? (
<FilesView <FilesView
files={files} files={files}

View File

@@ -261,17 +261,19 @@ export function ReviewTab({ initiativeId }: ReviewTabProps) {
)} )}
</div> </div>
{/* Right: Sidebar */} {/* Right: Sidebar — sticky so icon strip stays visible */}
<div className="border-l border-border p-4"> <div className="border-l border-border">
<ReviewSidebar <div className="sticky top-0 h-[calc(100vh-12rem)]">
files={allFiles} <ReviewSidebar
comments={comments} files={allFiles}
onFileClick={handleFileClick} comments={comments}
selectedCommit={selectedCommit} onFileClick={handleFileClick}
activeFiles={files} selectedCommit={selectedCommit}
commits={commits} activeFiles={files}
onSelectCommit={setSelectedCommit} commits={commits}
/> onSelectCommit={setSelectedCommit}
/>
</div>
</div> </div>
</div> </div>
)} )}