Skip to content

Improve the chat input field - #420

Merged
TheGreatAxios merged 6 commits into
corbitsdev:mainfrom
pratikbuilds:fix/composer-layout
Aug 30, 2026
Merged

TheGreatAxios merged 6 commits into
corbitsdev:mainfrom
pratikbuilds:fix/composer-layout

Conversation

@pratikbuilds

@pratikbuilds pratikbuilds commented Aug 27, 2026 •

Copy link
Copy Markdown
Contributor

Before

The composer felt like a thin, detached text field. Typing a message or using commands introduced excess vertical space and pushed the controls away from the draft, so the whole surface looked unbalanced.

Composer before

After

The input is now one full-width, bordered surface: the draft stays at the top and the actions remain anchored in a compact bottom rail. The keyboard shortcut appears only for a focused, non-empty draft, without creating extra padding. Attachment and send controls stay icon-only.

Composer after

Verification

  • bun test test/composer.test.tsx && bun run typecheck in packages/chat-ui
  • bun run build in apps/web
  • Browser-verified the focused, typed composer

bun run check is still blocked by an existing Hono 4.13.2/4.13.3 type mismatch in unchanged packages; dependency resolution was deliberately left out of this UI contribution.

@pratikbuilds pratikbuilds changed the title Keep composer controls in the action rail Improve the chat input field Aug 27, 2026
@TheGreatAxios
TheGreatAxios self-requested a review August 27, 2026 14:14
@TheGreatAxios TheGreatAxios added enhancement New feature or request ui labels Aug 27, 2026
@pratikbuilds

Copy link
Copy Markdown
Contributor Author

Fresh browser capture of the current composer state after the latest input-surface refinement.
clipboard

@TheGreatAxios TheGreatAxios 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.

critique · no blockers

The composer is now a full-width bordered column: draft on top, attach / Enter-to-send / send ArrowUp in a bottom rail. Hover-edit setText, Enter send, attachments, and send-disabled-while-sending all survive the merge onto current main. chat-ui composer tests pass.

Should-fix that landed on this branch before merge: the keyboard hint was opacity-only, so screen readers still announced “Enter to send” on an empty composer. Restored visibility: hidden plus aria-hidden until the draft is focused and non-empty.

Visual notes (not blocking): --ui-radius-lg is 0 in product so the box stays square; attach/send stay pills; focus is a --primary color swap rather than the old foreground inset hairline. Those match first-run composer more than the previous conversation row.

CI on this updated head (main merged in) is the merge gate.

@TheGreatAxios
TheGreatAxios merged commit 00bab80 into corbitsdev:main Aug 30, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants