Skip to content

Conversation

brandonskiser
Copy link
Contributor

@brandonskiser brandonskiser commented Apr 7, 2025

Description of changes:

  • Updating the help messaging for entering newlines to prioritize ctrl+j since it seems to be more commonly supported - for instance, my default keybindings already have something set for Alt+Enter across different terminals.
Screenshot 2025-04-07 at 4 25 25 PM

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@brandonskiser brandonskiser marked this pull request as ready for review April 7, 2025 23:29
@brandonskiser brandonskiser requested a review from a team April 7, 2025 23:29
@codecov-commenter
Copy link

codecov-commenter commented Apr 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 13.69%. Comparing base (a568960) to head (ca95236).
Report is 266 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1129   +/-   ##
=======================================
  Coverage   13.69%   13.69%           
=======================================
  Files        2361     2361           
  Lines      203706   203748   +42     
  Branches   184070   184112   +42     
=======================================
+ Hits        27889    27912   +23     
- Misses     174429   174447   +18     
- Partials     1388     1389    +1     

☔ View full report in Codecov by Sentry.
📢 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.

@brandonskiser brandonskiser enabled auto-merge (squash) April 7, 2025 23:35
@GoodluckH
Copy link
Contributor

GoodluckH commented Apr 7, 2025

Is ctrl+j also a popular combo on Mac? I think "command+key" is more popular

@brandonskiser
Copy link
Contributor Author

Is ctrl+j also a popular combo on Mac? I think "command+key" is more popular

you mean like "command+j"? That keybinding doesn't work, has to be control(^)

@@ -119,7 +119,7 @@ const WELCOME_TEXT: &str = color_print::cstr! {"
<em>/help</em> <black!>Show the help dialogue</black!>
<em>/quit</em> <black!>Quit the application</black!>

<cyan!>Use Alt(⌥) + Enter(⏎) to provide multi-line prompts.</cyan!>
<cyan!>Use Ctrl(^) + j to provide multi-line prompts.</cyan!>
Copy link
Contributor

@mschrage mschrage Apr 7, 2025

Choose a reason for hiding this comment

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

Suggested change
<cyan!>Use Ctrl(^) + j to provide multi-line prompts.</cyan!>
<cyan!>Tip: Use Ctrl(^) + j to add a newline in your prompt.</cyan!>

@brandonskiser brandonskiser merged commit 4307d77 into main Apr 7, 2025
20 checks passed
@brandonskiser brandonskiser deleted the bskiser/newline-messaging branch April 7, 2025 23:46
This was referenced Apr 30, 2025
mangaohua pushed a commit to mangaohua/amazon-q-developer-cli that referenced this pull request Jun 17, 2025
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.

4 participants