Skip to content

fix: always show Sign In button in both light and dark modes#181

Merged
texture-fleet-agent[bot] merged 2 commits intomainfrom
anvil/fix-signin-visibility
Apr 26, 2026
Merged

fix: always show Sign In button in both light and dark modes#181
texture-fleet-agent[bot] merged 2 commits intomainfrom
anvil/fix-signin-visibility

Conversation

@texture-fleet-agent
Copy link
Copy Markdown
Contributor

Summary

Sign In button was not visible in light mode on the mobile drawer. The root cause: button rendering was gated behind Clerk's isLoaded state (showAuth = mounted && isLoaded), and if Clerk was slow to initialize the button never appeared.

Changes

  • Removed showAuth gate entirely — Sign In now renders immediately regardless of Clerk loading state
  • Moved Sign In above Dark/GitHub row in mobile drawer so it's the first thing in the footer (no scrolling needed)
  • Simplified auth conditional to isSignedIn ? UserMenu : SignIn — cleaner, always visible
  • Applies to desktop nav, mobile top-bar, and mobile drawer

Before

Sign In button hidden in light mode until Clerk finishes loading (which may not happen quickly on mobile)

After

Sign In button always visible in both light and dark modes, prominently placed above utility buttons

Co-authored-by: Nicholas Brown nicholasalanbrown@gmail.com

- Remove showAuth gate — Sign In now renders immediately in both
  light and dark modes without waiting for Clerk isLoaded
- Move Sign In above Dark/GitHub row in mobile drawer so it's
  visible without scrolling
- Simplify auth conditional: isSignedIn ? UserMenu : SignIn
@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 26, 2026

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

Project Deployment Actions Updated (UTC)
commongrid Ready Ready Preview, Comment Apr 26, 2026 0:16am

Request Review

@texture-fleet-agent texture-fleet-agent Bot merged commit c4bf645 into main Apr 26, 2026
6 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