Skip to content

fix(web): return from footer pages to active thread - #6481

Open
edoedac0 wants to merge 1 commit into
pingdotgg:mainfrom
edoedac0:agent/fix-sidebar-back-navigation
Open

fix(web): return from footer pages to active thread#6481
edoedac0 wants to merge 1 commit into
pingdotgg:mainfrom
edoedac0:agent/fix-sidebar-back-navigation

Conversation

@edoedac0

@edoedac0 edoedac0 commented Aug 13, 2026

Copy link
Copy Markdown

Problem

Opening Usage or Pull Requests from an active thread replaces the footer actions with a Back button, but that button always navigated to /. This discarded the originating thread and showed the new-thread screen.

Fix

Use the router's in-app history state for the shared sidebar Back action. When history is available, the button returns to the route that opened the footer page; direct entries still fall back to /.

Impact

Users return to the active thread they came from after viewing Usage or Pull Requests. Direct links retain the existing safe fallback.

Validation

  • ./node_modules/.bin/vp run --filter @t3tools/web typecheck
  • Manually verified from an active thread in an isolated local T3 environment

Note

Fix footer back button to return to active thread using browser history

The footer back button in SidebarChromeFooter now calls window.history.back() when a browser history entry exists, falling back to navigating to / when there is no history. This uses the useCanGoBack hook from @tanstack/react-router to determine which path to take.

Macroscope summarized 2cfbf04.


Note

Low Risk
Small navigation-only change in the sidebar footer with a safe / fallback when history is empty.

Overview
The sidebar Back control on Usage and Pull Requests no longer always routes to /. SidebarChromeFooter now uses useCanGoBack from TanStack Router: when in-app history exists, handleBackClick calls window.history.back() so users return to the thread (or route) they came from; if there is no history (e.g. direct entry), it still navigates to /.

This matches the back-navigation pattern already used in settings-related screens.

Reviewed by Cursor Bugbot for commit 2cfbf04. Bugbot is set up for automated code reviews on this repo. Configure here.

@coderabbitai

coderabbitai Bot commented Aug 13, 2026

Copy link
Copy Markdown

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 Plus

Run ID: 0bd6d69b-7cc9-487b-b203-9a68fc8a6f8b

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

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.

@github-actions github-actions Bot added vouch:unvouched PR author is not yet trusted in the VOUCHED list. size:XS 0-9 changed lines (additions + deletions). labels Aug 13, 2026
@edoedac0
edoedac0 marked this pull request as ready for review August 13, 2026 15:15
@macroscopeapp

macroscopeapp Bot commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved 2cfbf04

Simple UX improvement that conditionally uses browser history navigation instead of always navigating to root. Uses standard tanstack/react-router hooks and browser APIs with existing fallback behavior preserved.

You can customize Macroscope's approvability policy. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS 0-9 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