Skip to content

Clicking logo now navigates to threads#1617

Merged
juliusmarminge merged 7 commits intopingdotgg:mainfrom
waLLxAck:codex/fix-logo-navigation-to-threads
Apr 2, 2026
Merged

Clicking logo now navigates to threads#1617
juliusmarminge merged 7 commits intopingdotgg:mainfrom
waLLxAck:codex/fix-logo-navigation-to-threads

Conversation

@waLLxAck
Copy link
Copy Markdown
Contributor

@waLLxAck waLLxAck commented Apr 1, 2026

What Changed

Made the sidebar T3 Code logo navigate back to the main threads screen.

The logo in the sidebar header is now a real button that routes to /, so clicking it from Settings or any other non-home view returns you to the threads landing screen. When already on the threads home screen, the click is a no-op.

Why

The current behavior is misleading: the logo looks clickable, but from Settings it does nothing. This is a small navigation bug in a high-visibility part of the UI.

This approach keeps the fix tight:

  • no route restructuring
  • no broader sidebar refactor
  • only the existing logo interaction is corrected

UI Changes

Before:
Clicking the sidebar logo from Settings did nothing.

After:
Clicking the sidebar logo returns the user to the main threads screen.

Video evidence:

  • Before: video showing Settings open and logo click doing nothing
  • After: video showing logo click returning to the threads screen

ezgif-23c0de6e1414b950

Checklist

  • This PR is small and focused
  • I explained what changed and why
  • I included before/after screenshots for any UI changes - N/A - Appearance remains the same
  • I included a video for animation/interaction changes

Note

Low Risk
Small UI/navigation tweak limited to the sidebar header; low risk aside from potential minor styling/accessibility regressions.

Overview
Makes the sidebar T3 Code wordmark actually navigate back to the threads landing route by replacing the non-interactive container with a Link to /.

Also adds an aria-label and focus/hover ring styling so the logo is keyboard-focusable and visually indicates interactivity.

Written by Cursor Bugbot for commit 70c66ce. This will update automatically on new commits. Configure here.

Note

Navigate to threads when clicking the sidebar logo

Replaces the non-interactive wordmark div in Sidebar.tsx with a Link component that routes to /. Adds an aria-label of "Go to threads" and focus/hover styles for accessibility.

Macroscope summarized 70c66ce.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 1, 2026

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f2698431-2746-4d74-886e-547570d0db0d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

@github-actions github-actions bot added size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Apr 1, 2026
@waLLxAck waLLxAck requested a review from juliusmarminge April 1, 2026 08:34
@juliusmarminge juliusmarminge enabled auto-merge (squash) April 2, 2026 21:42
@juliusmarminge juliusmarminge merged commit ae3ea39 into pingdotgg:main Apr 2, 2026
10 checks passed
gigq pushed a commit to gigq/t3code that referenced this pull request Apr 6, 2026
Chrono-byte pushed a commit to Chrono-byte/t3code that referenced this pull request Apr 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S 10-29 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants