Skip to content

[Fix] Shift+Enter sends chat message when Ctrl/Cmd+Enter mode is enabled#199

Merged
navedmerchant merged 2 commits into
mainfrom
fix/shift-enter-newline-chat-1cldtbtc6bsj9
May 19, 2026
Merged

[Fix] Shift+Enter sends chat message when Ctrl/Cmd+Enter mode is enabled#199
navedmerchant merged 2 commits into
mainfrom
fix/shift-enter-newline-chat-1cldtbtc6bsj9

Conversation

@roomote
Copy link
Copy Markdown
Contributor

@roomote roomote Bot commented May 19, 2026

Opened on behalf of Naved Merchant. View the task or mention @roomote for follow-up asks.

What problem this solves

Fixes an issue where users who enable Ctrl/Cmd+Enter to send can accidentally send a chat message with Shift+Enter instead of inserting a new line in the chat composer.

Why this change was made

The newline-mode keyboard handler treated Shift+Enter as a send shortcut even though that mode is meant to reserve sending for Ctrl/Cmd+Enter. This change narrows the send condition to Ctrl/Cmd+Enter and updates the regression test to lock that behavior in.

User impact

Users can keep using Shift+Enter to add a new line in the chat input, regardless of whether Ctrl/Cmd+Enter to send is enabled.

@roomote roomote Bot added the roomote:auto-resolve-conflicts Allow Roomote to auto-resolve merge conflicts for this PR label May 19, 2026
@roomote
Copy link
Copy Markdown
Contributor Author

roomote Bot commented May 19, 2026

2 checks still pending after the review wait window. See task

  • platform-unit-test (ubuntu-latest) is still in progress after the review wait window.
  • platform-unit-test (windows-latest) is still in progress after the review wait window.

@codecov
Copy link
Copy Markdown

codecov Bot commented May 19, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@navedmerchant navedmerchant marked this pull request as ready for review May 19, 2026 15:25
proyectoauraorg added a commit to proyectoauraorg/Zoo-Code that referenced this pull request May 19, 2026
Copy link
Copy Markdown
Contributor

@edelauna edelauna left a comment

Choose a reason for hiding this comment

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

Thanks, this always bugged me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

roomote:auto-resolve-conflicts Allow Roomote to auto-resolve merge conflicts for this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants