Skip to content

Conversation

@brendan-kellam
Copy link
Contributor

@brendan-kellam brendan-kellam commented Feb 11, 2026

This PR:

  • Removes the discord and github buttons from the top-right corner.
  • Moves the version indicator into the "what's new" menu.
  • Changes the design of the upper-right buttons. How they render is dependent on if the user is signed in or not.

Signed in:
image


image

Signed out:
image


image
image

Fixes #820

Summary by CodeRabbit

  • New Features

    • User avatar now shown in the top navigation for quick account access.
    • Added appearance settings menu (theme and code navigation).
    • New account dropdown with profile info and logout; sign-in button shown when not signed in.
  • Changed

    • Version indicator moved into the "What's New" dropdown and now shows build/commit link when available.
  • Removed

    • Discord and GitHub buttons removed from the top navigation.

@github-actions

This comment has been minimized.

@claude
Copy link

claude bot commented Feb 11, 2026

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@brendan-kellam
Copy link
Contributor Author

@coderabbitai review

@brendan-kellam brendan-kellam merged commit ac62f8f into main Feb 11, 2026
9 checks passed
@brendan-kellam brendan-kellam deleted the bkellam/login_button_redesign branch February 11, 2026 02:36
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

Caution

Review failed

The pull request is closed.

Walkthrough

Refactors top navigation by replacing SettingsDropdown with focused appearance and account controls, removes Discord/GitHub buttons, shows user avatar in the top bar, moves the version indicator into the "What's new" dropdown, and propagates auth sessions via async page/layout changes.

Changes

Cohort / File(s) Summary
Layout Restructuring
packages/web/src/app/[domain]/browse/layout.tsx, packages/web/src/app/[domain]/browse/layoutClient.tsx
Converted browse layout to async, await auth(), and delegate client UI to new LayoutClient which composes BrowseStateProvider, TopBar, FileTreePanel, resizable panels, and BottomPanel.
Session Propagation
packages/web/src/app/[domain]/chat/[id]/page.tsx, packages/web/src/app/[domain]/search/page.tsx, packages/web/src/app/[domain]/search/components/searchResultsPage.tsx
Pages now await auth() and pass `session: Session
Appearance Components
packages/web/src/app/[domain]/components/appearanceDropdownMenu.tsx, packages/web/src/app/[domain]/components/appearanceDropdownMenuGroup.tsx
Added new appearance dropdown and grouped menu for theme (Light/Dark/System) and code navigation (Default/Vim).
User Account Control
packages/web/src/app/[domain]/components/meControlDropdownMenu.tsx
Added MeControlDropdownMenu rendering avatar trigger, user header, appearance group, settings link, and sign-out logic.
TopBar & Navigation Updates
packages/web/src/app/[domain]/components/topBar.tsx, packages/web/src/app/[domain]/components/navigationMenu/index.tsx, packages/web/src/app/[domain]/components/settingsDropdown.tsx
TopBar now accepts session prop and conditionally renders MeControlDropdownMenu when signed in or login button + AppearanceDropdownMenu when not; removed SettingsDropdown (file deleted) and eliminated Discord/GitHub action forms.
What's New Enhancement
packages/web/src/app/[domain]/components/whatsNewIndicator.tsx
Moved current version and optional commit link into the WhatsNewIndicator popover; set default props for newsItems and autoMarkAsRead.
Changelog
CHANGELOG.md
Documented unreleased changes: avatar in top-bar, version indicator moved, and removal of Discord/GitHub buttons.

Sequence Diagram

sequenceDiagram
    participant Browser as Client Browser
    participant Page as Page/Layout (server)
    participant Auth as Auth (next-auth)
    participant UI as Client UI (LayoutClient / TopBar / Menus)

    Browser->>Page: Request page
    Page->>Auth: await auth()
    Auth-->>Page: session (or null)
    Page->>Browser: render with session prop

    Browser->>UI: Initialize client components (LayoutClient)
    UI->>UI: TopBar receives session
    alt session != null
        UI->>UI: Render MeControlDropdownMenu (avatar, account menu)
        UI->>UI: Render Appearance controls within menu
    else session == null
        UI->>UI: Render Login button + AppearanceDropdownMenu
    end
    UI-->>Browser: Render final UI
Loading

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

Suggested labels

sourcebot-team

Suggested reviewers

  • msukkari
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bkellam/login_button_redesign

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 11, 2026

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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.

[FR] EE: Option to disable Discord button, Sourcebot repo and Ask components in the homepage

1 participant