Skip to content

feat(web): middle-click opens links in the integrated browser - #1

Merged
n0one42 merged 3 commits into
mainfrom
feat/middle-click-integrated-browser
Nov 2, 2026
Merged

n0one42 merged 3 commits into
mainfrom
feat/middle-click-integrated-browser

Conversation

@n0one42

@n0one42 n0one42 commented Oct 31, 2026 •

Copy link
Copy Markdown
Owner

Personal fork draft for review. Not an upstream pingdotgg/t3code submission; no upstream maintainer has approved this feature.

Problem

With Open links in set to the system browser, opening one chat or pull request link beside the thread needs the context menu's Open in integrated browser. Middle-click and Cmd/Ctrl-click both go external.

Change

Middle-click opens HTTP(S) links in the integrated browser when available. Plain clicks keep their settings. Cmd/Ctrl-click, including with middle-click, still opens the system browser.

  • Pull request controls (single-PR badge, PR list, sidebar row, pull request panel) open the PR page in the integrated browser. Middle-clicking another thread's sidebar PR activates that thread.
  • Thread server rows: plain click already opened the panel; middle-click now does too, for consistency only.
  • Setting description and docs/user/source-control.md updated.

In supported desktop preview tabs, the existing Annotate preview can then select an element. Unchanged: terminal links and paste, mobile, preview permissions, fallbacks. No new setting; the normal-click default is unchanged.

Scope and approval

This reverses the middle-click behavior chosen in pingdotgg#19854 and pingdotgg#26267. #22007 was closed because the setting and context menu exist (comment); this adds a per-link shortcut without changing the normal-click default. Direction and scope approval is pending in the upstream Ideas discussion. This fork draft remains separate from any upstream PR.

In my use of the Codex app, middle-click opens its built-in browser. This comparison is a personal observation, not a verified documentation claim.

Verification

  • Targeted typecheck and lint of the changed web files passed.
  • The existing browserLinkTarget test (non-web schemes stay external) passes; it has no mouse-button cases, so the gesture lacks automated coverage.
  • vp run check:changed upstream/main: passed on fbf4a2532d71 against 5fe18d79ac83.
  • A packaged fork build contains the change; packaging checks do not prove click behavior. Informal feedback on a test link is not recorded mouse evidence.

Pending: a mouse pass over chat links, PR controls, server rows, fallback, app.t3.codes and terminal paste; before/after screenshots; a recording.

Implementation: GPT-6.1 Sol via T3 Code (Codex). PR text: Claude Opus 5.5 via T3 Code (Claude Code).

@n0one42
n0one42 marked this pull request as ready for review November 1, 2026 09:50
@n0one42
n0one42 merged commit c744954 into main Nov 2, 2026
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