Skip to content

fix: composer ui in mobile#2377

Merged
steve8708 merged 2 commits into
BuilderIO:mainfrom
shivanshu11092003:fix-composer-ui
Jul 24, 2026
Merged

fix: composer ui in mobile#2377
steve8708 merged 2 commits into
BuilderIO:mainfrom
shivanshu11092003:fix-composer-ui

Conversation

@shivanshu11092003

@shivanshu11092003 shivanshu11092003 commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

fix mobile chat composer's control row to match our standard Claude composer.

Before

image

After

image image

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@netlify

This comment has been minimized.

@github-actions

github-actions Bot commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Visual recap — skipped

The visual recap job did not run for this pull request. This is informational only and does not block the PR.

Recap skipped for ae3c525: external fork PR requires a maintainer to apply the recap label to the current head SHA.

builder-io-integration[bot]

This comment was marked as outdated.

@builder-io-integration builder-io-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Builder reviewed your changes and has a few items to flag 🟡

Review Details

Incremental Code Review Summary

The latest commit addresses the previous review’s multi-file attachment issue: the picker still permits multiple files, but all returned assets are now converted with Promise.all and appended through addAttachments. No prior review thread requires resolution because that finding was not posted as an inline comment.

New Finding

🟡 MEDIUM: The redesigned bottom control row is a single non-wrapping flex row containing the plus control, model selector, mode selector, microphone, and send/stop control. On narrow supported devices (around 320px wide), a selected model/effort label can consume enough width that the rightmost send/stop control is pushed beyond the composer’s available width. The model/mode controls should be allowed to shrink/ellipsize or the row should reflow while preserving send/stop access.

The Expo dependency remains aligned with SDK 57, and the updated attachment conversion path is sound.

🧪 Browser testing: Skipped — infra-unavailable (the native Expo screen is not reachable from the running web dev server).

@steve8708

Copy link
Copy Markdown
Contributor

MUCH better, thanks @shivanshu11092003 !

@steve8708
steve8708 merged commit 21c2c1d into BuilderIO:main Jul 24, 2026
27 checks passed
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.

2 participants