tas/Codewalkers
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

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

Browse Source
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
Download Patch File Download Diff File Expand all files Collapse all files

2
apps/web/src/components/review/ReviewSidebar.tsx
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}

6
apps/web/src/components/review/ReviewTab.tsx
View File

@@ -261,8 +261,9 @@ 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">
<div className="sticky top-0 h-[calc(100vh-12rem)]">
<ReviewSidebar <ReviewSidebar
files={allFiles} files={allFiles}
comments={comments} comments={comments}
@@ -274,6 +275,7 @@ export function ReviewTab({ initiativeId }: ReviewTabProps) {
/> />
</div> </div>
</div> </div>
</div>
)} )}
</div> </div>
); );