Skip to content

pdf viewer sidebar-aware rounding and small polish#770

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

pdf viewer sidebar-aware rounding and small polish#770
moehdotdev merged 1 commit into
mainfrom
home-uiux-changes

Conversation

@moehdotdev

Copy link
Copy Markdown
Member

main changes

before :
image
image
image

now :
image
image

pdf viewer adapts to sidebar state

  • PdfViewerShell and PdfViewerContent now read { open, isMobile } from useSidebar and apply app-radius-lg only when the sidebar is open and not on mobile so the viewer corner rounds when the sidebar is expanded (matching the main content area) and goes flush when it's collapsed or on mobile
  • same logic applied to the toolbar pill inside PdfViewerContent

sidebar trigger hidden on pdf routes

  • SidebarTrigger in HomeClientLayout is now suppressed on pdf routes (!isPdfRoute) since the pdf viewer renders its own sidebar-aware chrome and the trigger in the global header was overlapping it

move dialogs toast action button

  • MoveNoteDialog and MovePdfDialog success toast action button padding changed from px-4 to px-3 h-8, and link text gets text-xs in MoveNoteDialog fits better inside the compact toast layout

toast close button

  • close button position tightened from right-1 top-1 p-1 to right-0.5 top-0.5 p-0.5 so it sits closer to the corner
  • toast right padding bumped from pr-6 to pr-7 to compensate and prevent content from running under the close button

@vercel

vercel Bot commented May 25, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
notevo Ready Ready Preview, Comment May 25, 2026 5:20pm

@moehdotdev moehdotdev merged commit bd02299 into main May 25, 2026
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