fix: Fix sidebar layout — sticky height constraint, padding on content only
This commit is contained in:
@@ -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}
|
||||
|
||||
@@ -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>
|
||||
);
|
||||
|
||||
Reference in New Issue
Block a user