Skip to content

fix(web): prevent Enter key from sending message during IME composition - #715

Merged
zmanian merged 2 commits into
nearai:mainfrom
ygcedu:feature/main
Mar 8, 2026
Merged

zmanian merged 2 commits into
nearai:mainfrom
ygcedu:feature/main

Conversation

@ygcedu

@ygcedu ygcedu commented Mar 8, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions Bot added scope: channel/web Web gateway channel size: XS < 10 changed lines (excluding docs) risk: medium Business logic, config, or moderate-risk modules contributor: new First-time contributor labels Mar 8, 2026

@zmanian zmanian left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Clean fix. e.isComposing is the standard API for handling IME composition — well-supported across all modern browsers (Chrome 56+, Firefox 31+, Safari 10.1+). This prevents Enter from submitting mid-composition for CJK input users.

@zmanian
zmanian enabled auto-merge (squash) March 8, 2026 19:54

@zmanian zmanian left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM - IME composition fix for web gateway.

@zmanian
zmanian merged commit da2569b into nearai:main Mar 8, 2026
27 checks passed
bkutasi pushed a commit to bkutasi/ironclaw that referenced this pull request Mar 28, 2026
…on (nearai#715)

Co-authored-by: Zaki Manian <zaki@iqlusion.io>
drchirag1991 pushed a commit to drchirag1991/ironclaw that referenced this pull request Apr 8, 2026
…on (nearai#715)

Co-authored-by: Zaki Manian <zaki@iqlusion.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor: new First-time contributor risk: medium Business logic, config, or moderate-risk modules scope: channel/web Web gateway channel size: XS < 10 changed lines (excluding docs)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants