fix(mobile): match dashed thought row box to solid tool row box - #5318
Merged
Conversation
The dashed FixedPartRow used rounded-xl and a 1.5px dashed stroke, which receded the visible ink from the layout edge on iOS and made a collapsed thought pill look farther from neighboring tool pills than those pills look from each other. Reuse the solid outer box (overflow-hidden rounded-lg border) and keep only border-dashed, so the visible gutter matches the gap-2 rhythm.
Contributor
Code Review SummaryStatus: No Issues Found | Recommendation: Merge Executive SummaryReviewed the mobile Files Reviewed (2 files)
Reviewed by kimi-k3 · Input: 29.3K · Output: 2.7K · Cached: 130.6K Review guidance: REVIEW.md from base branch |
eshurakov
approved these changes
Aug 18, 2026
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
2 participants
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.
fix(mobile): match dashed thought row box to solid tool row box
What changed
A collapsed thought pill in the mobile cloud-agent transcript now sits the same visible distance from neighboring tool pills as two tool pills sit from each other.
User note
The thought pill between two tool rows no longer looks farther away than the tool rows look from each other. The dashed outline and the
THOUGHT/THINKINGlabel are unchanged.Product note
The transcript row rhythm is now uniform: the dashed thought card reuses the solid tool card's radius, border width, and overflow clip, so the visible gutter matches the
gap-2spacing between tool rows.Maintainer note
FixedPartRowdrew two card shapes: the solid tool card usedoverflow-hidden rounded-lg border border-border, while the dashed thought card usedrounded-xl border-[1.5px] border-dashed border-border. The thicker stroke and larger radius receded the visible ink from the layout edge on iOS. The dashed branch now reuses the solid outer box and keeps onlyborder-dashed. A mounted test locks the dashed outer class.Human steps
No human step is needed.
Visual Changes
This is a subtle 1px spacing adjustment (the dashed thought row now uses the solid row's
rounded-lgradius and 1px border instead ofrounded-xland a 1.5px stroke). No screenshot is attached; the change is locked by a mounted class test and the iOS regression check passed.