Repository navigation
fix(remote): put every phone chrome band on the 12px feed gutter - #1023
Merged
Merged
Conversation
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
The v1 phone chrome was born with a 14px gutter while the mounted desktop feed, the composer and the v2 chrome use 12px. The 2px shear between adjacent bands was the owner's 'mobile padding is broken' report. Five bands move to 12px, and a stylesheet-text contract test pins the single gutter. It fails on 4 bands against main. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Juliusolsson05
added a commit
that referenced
this pull request
Sep 19, 2026
Pick up #1022 (goal-loop input freeze), #1023, #933 (tmux inventory) and #1012. One conflict, in useKeybinds' onBlur: keep #1022's dismissGoalLoop() and drop `pendingTiledResizeIndex`, which #992 deleted along with tiled resize. #1022's overlay gate and rebind-safe dismiss chord merged intact. Verified: tsc -b clean. goal-loop + tile-tree + tldr renderer 175/175; tmux + storage + remote-client unit 112/112. That includes #933's real-workspace recovery test running against #992's model. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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.
Problem
The owner reported that margin and padding on mobile are broken. In the phone (remote-client) shell, the v1 chrome bands (
.topbar,.section-label,.session-row,.terminal,.working) used a 14px horizontal gutter. The mounted desktop feed (--feed-gutter: 12px), the composer and the v2 chrome all use 12px, so adjacent bands sat 2px apart.This work was finished by an earlier agent session and left uncommitted in the main checkout. This PR moves it onto a branch, unchanged (release-readiness thread T3).
Implemented
gutterContract.test.tsasserts the stylesheet text, which is the file the phone loads. jsdom never evaluates this CSS, so computed-style assertions could not see it;nordTheme.test.tsuses the same approach.Verification
.topbar,.section-label,.session-rowand.terminalfail.🤖 Generated with Claude Code