Skip to content

fix(web): server rows keep their source visible, cmd-click opens your browser - #26267

Merged
t3dotgg merged 1 commit into
mainfrom
t3/servers-panel-redesigns
Oct 28, 2026
Merged

t3dotgg merged 1 commit into
mainfrom
t3/servers-panel-redesigns

Conversation

@t3dotgg

@t3dotgg t3dotgg commented Oct 28, 2026

Copy link
Copy Markdown
Member

The Servers panel cut off where a server came from ("Agent · node-Main…"), and hovering a row hid that text behind the Stop button. You also could not open a server in your own browser from there.

Now each row has two lines: localhost:<port> on top, and the source and process below it with the full row width. The text keeps space clear for the Stop button, so hover no longer hides anything. Cmd-click (Ctrl-click off Mac) or middle-click opens the server in the system browser, like PR links do. A plain click still opens the browser panel. The external URL goes through resolveDiscoveredServerUrl, so remote and tunnel clients get an address they can reach.

Before (hover) After After (hover)
before after after hover

Proof: in a dev app with a server running in a thread terminal, Ctrl-click and middle-click called window.open("http://localhost:47123/", "_blank") and left the browser panel closed. Right-click did nothing, and a plain click opened the browser panel. vp run check:changed passes.

Reviewed with sol-loop: 1 round with GPT-6.1 Sol. Sol verdict: approved

Created with Claude Opus 5.5 in T3 Code (Claude Code harness).

🤖 Generated with Claude Code

… browser

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:M 30-99 changed lines (additions + deletions). labels Oct 28, 2026
@macroscopeapp

macroscopeapp Bot commented Oct 28, 2026

Copy link
Copy Markdown
Contributor

Approvability

Verdict: Approved at 037689e

Macroscope's review found this PR approvable — Server rows keep their source visible and support explicit modified or middle clicks to open the system browser. Ordinary clicks and stop controls retain their existing behavior, keeping the change narrowly scoped.

You can add or adjust custom eligibility rules. Learn more.

@t3dotgg
t3dotgg merged commit 8947f3e into main Oct 28, 2026
29 of 30 checks passed
@t3dotgg
t3dotgg deleted the t3/servers-panel-redesigns branch October 28, 2026 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M 30-99 changed lines (additions + deletions). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant