fix: Make review sidebar sticky to viewport with internal scrolling
Removed overflow-hidden from grid container and changed sidebar to sticky positioning with viewport-relative max-height. Sidebar content scrolls independently while staying visible during diff navigation.
This commit is contained in:
@@ -333,10 +333,10 @@ export function ReviewTab({ initiativeId }: ReviewTabProps) {
|
|||||||
/>
|
/>
|
||||||
|
|
||||||
{/* Main content area — sidebar always rendered to preserve state */}
|
{/* Main content area — sidebar always rendered to preserve state */}
|
||||||
<div className="grid grid-cols-1 lg:grid-cols-[260px_1fr] overflow-hidden rounded-b-lg">
|
<div className="grid grid-cols-1 lg:grid-cols-[260px_1fr] rounded-b-lg">
|
||||||
{/* Left: Sidebar — sticky so icon strip stays visible */}
|
{/* Left: Sidebar — sticky to viewport, scrolls independently */}
|
||||||
<div className="border-r border-border">
|
<div className="border-r border-border">
|
||||||
<div className="sticky top-0 h-[calc(100vh-12rem)]">
|
<div className="sticky top-0 h-screen max-h-[calc(100vh-4rem)] overflow-hidden">
|
||||||
<ReviewSidebar
|
<ReviewSidebar
|
||||||
files={allFiles}
|
files={allFiles}
|
||||||
comments={comments}
|
comments={comments}
|
||||||
|
|||||||
Reference in New Issue
Block a user