Skip to content

Put the reply bar's expand button inside the field on iOS - #97

Merged
simonhamp merged 1 commit into
mainfrom
fix/reply-expand-in-field
Oct 27, 2026
Merged

simonhamp merged 1 commit into
mainfrom
fix/reply-expand-in-field

Conversation

@simonhamp

Copy link
Copy Markdown
Member

On iOS the reply field on a commit page is a Liquid Glass plate, and the expand button sat outside it on the left. iOS puts a field's own controls inside the field at its trailing edge, so the button now lives there. It stays at the bottom of the plate as the draft grows to several lines.

Android is unchanged: the button still leads the bar. Its after shots are pixel-identical to the before shots apart from the system gesture bar.

Things to look at:

  • The glass and rounded corners moved from reply-field to a new reply-plate row that holds the field and the button. reply-field keeps the @pressUp that tells the page you're replying, and the button sits beside it rather than inside it, so pressing expand doesn't also fold away the earlier commits.
  • Placement is decided with Platform::isIos() in the view, since it's a different tree on each platform rather than a different class.
  • The two Liquid Glass tests now check reply-plate, and a new test checks where the button sits on each platform.

Screenshots

iOS light iOS dark Android light Android dark
Before
After

A long draft on iOS, with the button staying at the bottom of the field:

  • No visible change

🤖 Generated with Claude Code

On iOS the reply field is a Liquid Glass plate, and a field's own
controls belong inside it at the trailing edge, so the expand button now
sits there and stays at the bottom as the draft grows. It's beside the
field's press target rather than in it, so pressing it doesn't fold the
earlier commits away as a tap on the field does. Android keeps it ahead
of the field.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@simonhamp
simonhamp merged commit d37f21a into main Oct 27, 2026
2 checks passed
@simonhamp
simonhamp deleted the fix/reply-expand-in-field branch October 27, 2026 14:38
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