Skip to content

Show sidebar trigger only when sidebar is collapsed#215

Merged
BunsDev merged 1 commit into
mainfrom
okcode/persistent-sidebar-trigger
Apr 2, 2026
Merged

Show sidebar trigger only when sidebar is collapsed#215
BunsDev merged 1 commit into
mainfrom
okcode/persistent-sidebar-trigger

Conversation

@BunsDev

@BunsDev BunsDev commented Apr 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Make the sidebar trigger visible on desktop when the sidebar is collapsed, instead of hiding it behind the mobile breakpoint.
  • Keep the trigger hidden while the desktop sidebar is expanded so the sidebar rail remains the primary toggle.
  • Apply the same trigger behavior to chat, settings, home, and project subpage headers for consistent navigation.

Testing

  • Not run (PR description only).
  • Checked the diff for the trigger visibility changes across all affected headers.
  • Verified the sidebar trigger now conditionally hides on desktop based on sidebar open state rather than viewport width.

- Keep the trigger visible on desktop only when the sidebar is closed
- Remove page-level mobile-only trigger hiding for chat, settings, and review shells
@vercel

vercel Bot commented Apr 2, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
okcode-marketing Ready Ready Preview Apr 2, 2026 9:28pm
v0-compute-the-platform-to-build Ready Ready Preview, Open in v0 Apr 2, 2026 9:28pm

@github-actions github-actions Bot added the size:S label Apr 2, 2026
@github-actions github-actions Bot added the vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. label Apr 2, 2026
@BunsDev BunsDev merged commit c40ad5e into main Apr 2, 2026
9 of 10 checks passed
@BunsDev BunsDev deleted the okcode/persistent-sidebar-trigger branch April 6, 2026 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant