Repository navigation
Conversation
Contributor
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — This is a small, contained dark-mode presentation fix: switched-off environment identities remain dimmed while their controls stay distinguishable, and unchecked dark-mode switch thumbs gain contrast. No state, persistence, API, or workflow behavior changes. Notes:
You can add or adjust custom eligibility rules. Learn more. |
macroscopeapp
Bot
dismissed
their stale review
October 3, 2026 03:18
Dismissing prior approval to re-evaluate 73a6984
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #14979.
In dark mode an unchecked switch is close to invisible. Its thumb is
bg-background, the same color as the page, and the track is an 8% white fill. On Settings → Connections it gets worse. A switched-off environment row is drawn at 60% opacity, switch included, so an enabled-but-off switch looks the same as a disabled one. The reporter read both saved environments as unavailable and lost track of their remote threads.The fix has two parts:
Switch: an unchecked thumb in dark mode usesmuted-foreground, the theme's secondary text gray, instead of the page color. Light mode, the checked state, and the disabled opacity are unchanged. fix(web): make the unchecked switch thumb visible in dark mode #10477 tried aforeground(near-white) thumb and was closed because it pulled too much attention. This uses the same gray as the row's "Off" text, so the knob reads as a control without adding white to dark surfaces.EnvironmentRow: a switched-off row now dims only the icon, name, and subtitle. The switch and menu stay at full strength, so "off" and "unavailable" look different.Dark theme, web dev server, Chrome:
Reproduced with a worktree dev server and a second local server paired as a remote environment, then switched off. After the change I checked the same row switched on (unchanged blue switch) and the disabled "Network access" switch (still dimmed by its own disabled opacity).
vp fmt --checkandvp linton both files andvp run --filter @t3tools/web typecheckpass. Desktop renders the same web UI. Mobile uses its own native switch and is not affected. I did not check custom app themes.Created with Claude Opus 5.5 in Claude Code.
Fixes #17874.