Skip to content

fix(chat): align mobile controls and suppress steering completion warning - #3221

Merged
bobleer merged 1 commit into
GCWing:mainfrom
bobleer:bob/fix-steering-presentation
Sep 24, 2026
Merged

bobleer merged 1 commit into
GCWing:mainfrom
bobleer:bob/fix-steering-presentation

Conversation

@bobleer

@bobleer bobleer commented Sep 24, 2026

Copy link
Copy Markdown
Collaborator

Changes

Treat user_steering as a normal completion in the desktop transcript, so handing off to a queued user message no longer displays the generic abnormal-ending warning. Existing failure notices remain unchanged.

Align mobile assistant prose with thinking/tool rows. Reduce send, stop, and attachment controls from 48px to 44px; use equal composer insets and align queue info/remove controls with the stop button in both expanded and collapsed layouts.

Validation

  • Desktop completion notice tests: 9 passed, including steering with/without a final response.
  • Real Chrome mobile queue browser tests: 3 passed, including action behavior, queue placement, button centers, equal collapsed-composer insets, and prose alignment.
  • Checked 320/390/768/1200px layouts, keyboard-height viewport, dark/light themes; inspected screenshots.
  • pnpm run check:web, mobile type-check/build, and pnpm run theme:color-audit:all passed.

Remote control presentation was exercised with real-browser fixtures and simulated host/relay IO. No live-device, SSH workspace, Peer Device, or Detached Dispatch end-to-end run; no transport, runtime, or persisted-shape changes.

@bobleer
bobleer merged commit 1058934 into GCWing:main Sep 24, 2026
9 checks passed
@bobleer
bobleer deleted the bob/fix-steering-presentation branch September 24, 2026 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant