Repository navigation
Conversation
ApprovabilityVerdict: Approved at Macroscope's review found this PR approvable — Removes an unnecessary inset from mobile thread host metadata so it aligns with the row edge, while leaving title/search alignment unchanged. The change is limited to a small, self-contained visual correction with no broader runtime or data impact. Notes:
No code changes detected at You can add or adjust custom eligibility rules. Learn more. |
Walkthrough
Merge Risk:
|
Dismissing prior approval to re-evaluate 6ad47c8
6ad47c8 to
17ccd33
Compare
|
@maria-rcks btw @juliusmarminge wanted to ponder this layout more so hold off on any auto merges. |
|
Note Written by thanks @PixPMusic. your one-line metadata inset removal already is the smallest fix. i kept your commit and implementation intact and merged main verification:
holding for julius's layout decision as you requested. this pr remains open and has not been merged. |
…ingdotgg#25669) Cherry-picked from pingdotgg#25669 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Note Written by Verified in a real dev environmentNo changed behavior could be exercised on this Linux Boat VM. This is an unverifiable result, not a runtime pass. I read the description, discussion, commit authors, and exact one-line diff at head Device preflight returned: The repository mobile verification skill requires stopping verification when device access is unavailable. Chrome cannot exercise this React Native component. No source changes, commits, pushes, or new dev servers were made. Existing CI results are not evidence of native behavior. Not verifiedMain reproduction and current-head iOS/Android rendering, Home and sidebar host alignment, error/PR/provider metadata, search excerpt alignment, large text, and row navigation remain unverified. No native client is accessible here, so I did not create stand-in web evidence or claim a Codex turn would verify this layout. No new screenshots or recordings were captured. Main/PR installation and server startup were skipped after the native-device blocker was established. To complete verification, enable a native device host and compare main and this exact head using app-created project/thread data and real Codex |


Problem
#25415 added a project-icon-sized inset to the mobile thread metadata. This shifts the machine indicator and host label away from the row's leading edge and wastes horizontal space on narrow screens.
Change
Remove the metadata inset so the machine indicator lines up with the project's leading edge. Keep search excerpts aligned with title text. Icon sizing and alignment from #25668 are now merged into main and included in this PR’s base.
Scope and approval
Alex requested this focused visual correction. It applies to shared iOS/Android Home and sidebar card rows, including the error/PR/provider metadata line. No provider, contract, transport, web/desktop, pending-draft, or slim-row behavior changes.
Verification
5127de8c29ed, including merged icon scaling fix(mobile): scale and align thread list icons #25668. Current head:17ccd33141. The remaining diff removes only the metadata inset.vp run check:changedpassed formatting, lint, knip, and affected mobile/scripts typechecks after the rebase. All 8 required CI checks passed; CodeRabbit found no actionable issues and Macroscope approved this head.Model: GPT-6.1 Sol (medium reasoning) | Harness: Codex in T3 Code