Skip to content

feat: implemented Ctrl+Enter keyboard shortcut for prompt textarea#12

Merged
nive62tech merged 1 commit into
nive62tech:mainfrom
reory:feat/keyboard-shortcut
Jun 17, 2026
Merged

feat: implemented Ctrl+Enter keyboard shortcut for prompt textarea#12
nive62tech merged 1 commit into
nive62tech:mainfrom
reory:feat/keyboard-shortcut

Conversation

@reory

@reory reory commented May 26, 2026

Copy link
Copy Markdown
Contributor

Description

Closes #8

This PR addressed the issue where the Ctrl+Enter shortcut was not firing while typing
inside the prompt text box.

Changes Implemented

  • Updated frontend/app.js to target the prompt element directly instead of
    listening globally on the document.
  • Added e.preventDefault() to stop the browser from inserting an unwanted new
    line inside the textbox area when the shortcut is used.
  • Added descriptive comments for future maintainers.

Testing features

  • Verified locally that Ctrl+Enter successfully triggers the evaluation.
  • Verified that pressing Enter by itself still correctly creates a new line.

@reory

reory commented Jun 11, 2026

Copy link
Copy Markdown
Contributor Author

hi @nive62tech
Just dropping a gentle note on this one. Its been a couple of weeks.
The PR cleanly fixes the Ctrl+Enter shortcut issue mentioned in #8 and there are no merge conflicts.
Whenever you have a spare moment to take a look please would be greatly appreciated.

Thank you for maintaining the project. 😊

@nive62tech nive62tech self-requested a review June 17, 2026 15:29
@nive62tech nive62tech merged commit dddd68f into nive62tech:main Jun 17, 2026
@nive62tech

Copy link
Copy Markdown
Owner

Great work @reory !
Love that you targeted the textarea specifically and added preventDefault — that's a much cleaner implementation. Merging this in!
Thank you for contributing

@reory

reory commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

@nive62tech Thank you 😊

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.

Add keyboard shortcut Ctrl+Enter

2 participants