Skip to content

feat(ai): send the Bob prompt with Enter - #1006

Merged
surajshetty3416 merged 2 commits into
frappe:developfrom
stravo1:bob-enter-to-send
Oct 29, 2026
Merged

surajshetty3416 merged 2 commits into
frappe:developfrom
stravo1:bob-enter-to-send

Conversation

@stravo1

@stravo1 stravo1 commented Oct 28, 2026

Copy link
Copy Markdown
Member

In the Bob AI panel, Enter now sends the prompt and Shift+Enter adds a new line. Before, only Cmd+Enter or Ctrl+Enter sent it.

  • The "/" command menu still gets the key first, so Enter picks a menu item while the menu is open.
  • Enter does nothing while an input method composes a word. Chinese, Japanese and Korean users can pick a word without sending a half-typed prompt.
  • Cmd+Enter and Ctrl+Enter still send.
  • The Send tooltip shows Enter instead of Mod+Enter.

Testing

Checked in the editor with Playwright: Enter in an empty prompt adds no new line, and Shift+Enter between two words gives two lines.

Enter sends the prompt and Shift+Enter adds a new line. Before, only
Cmd+Enter or Ctrl+Enter sent it. Enter does nothing while an input
method composes a word, so Chinese, Japanese and Korean users can pick
a word without sending a half-typed prompt. The "/" command menu still
gets the key first.

Co-Authored-By: Claude <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 28, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.04%. Comparing base (e9a6ac4) to head (3f635de).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1006      +/-   ##
===========================================
+ Coverage    81.01%   81.04%   +0.03%     
===========================================
  Files           66       66              
  Lines         7137     7139       +2     
===========================================
+ Hits          5782     5786       +4     
+ Misses        1355     1353       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

The manual key checks become the declarative modifier. .exact skips Enter
when any other system modifier is held, so Shift+Enter keeps the textarea
default (newline) with no handler of its own, and Cmd+Enter / Ctrl+Enter
still send.

isComposing stays in the handler: no modifier covers an IME composition,
and the Enter that picks a candidate arrives as key "Enter" with no
modifiers held, so .exact alone would send half-typed prompts from dead
keys, mobile keyboards and CJK input.
@surajshetty3416
surajshetty3416 merged commit d83526f into frappe:develop Oct 29, 2026
9 of 10 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