-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: multiline ai assist prompts #1346
Conversation
We require contributors to sign our Contributor License Agreement, and we don"t have one on file for you. In order for us to review and merge your code, please contact our team at https://www.quadratichq.com/contact. |
@AyushAgrawal-A2 is attempting to deploy a commit to the Quadratic Team on Vercel. A member of the Team first needs to authorize it. |
We require contributors to sign our Contributor License Agreement, and we don"t have one on file for you. In order for us to review and merge your code, please contact our team at https://www.quadratichq.com/contact. |
We require contributors to sign our Contributor License Agreement, and we don"t have one on file for you. In order for us to review and merge your code, please contact our team at https://www.quadratichq.com/contact. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a few additional fixes, tweaks to this, including:
- CMD+Enter submits the textarea, with Enter and Shift+Enter now inserting a line. This aligns the input more broadly with web app textarea inputs (like twitter) as well as our own code editor which uses CMD+Enter to "submit" and run the code while Enter just adds a new line. Added a shortcut to the tooltip to indicate this to the user
- Multi-line user inputs were not displaying as multi-line messages from the user who typed them. Now they do.
- "Send" button aligns to the bottom of the textarea as it expands in height (so does loading indicator)
Will merge this once all tests pass... |
closes #1210
Changes:
ai.assist.prompt.input.webm