Skip to content

pdf viewer toolbar positioning, gradient layering fixes#765

Merged
moehdotdev merged 1 commit into
mainfrom
home-uiux-changes
May 23, 2026
Merged

pdf viewer toolbar positioning, gradient layering fixes#765
moehdotdev merged 1 commit into
mainfrom
home-uiux-changes

Conversation

@moehdotdev

Copy link
Copy Markdown
Member

what changed

pdf viewer toolbar nudged down

  • toolbar container moved from top-1 to top-10 so it clears the app header bar and doesn't overlap the breadcrumb/settings row

gradient fade inside the pdf viewer

  • added a h-[4rem] gradient div at the top of the pages container (from-background via-background/75 to-transparent) so the toolbar floats visually above the pdf content without a hard edge
  • this gradient sits at z-[900000] inside the pages wrapper, above the pdf canvas layers

z-index fixes

  • global header (HomeClientLayout) bumped from z-50 to z-[60000] so it renders above the pdf viewer's internal layers, which were creeping up with higher z-values
  • scroll fade gradient in HomeClientLayout changed from z-20 to z-[50000] and its height reduced from 7rem to 4rem to match the tighter header height; gradient stops adjusted to from-0% via-45% for a slightly sharper fade
  • pdf route scroll container top padding changed from pt-12 to pt-0 since the header is now absolutely positioned above the viewer and doesn't need padding compensation

pdf viewer shell corner

  • rounded-tl-lg added to PdfViewerShell's root element so the viewer respects the same corner radius as the main content area when the sidebar is open

@vercel

vercel Bot commented May 23, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
notevo Building Building Preview, Comment May 23, 2026 6:19pm

@moehdotdev moehdotdev merged commit b90cfe7 into main May 23, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant