Skip to content
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

Indicate the multiline input with \n> #72

Merged
merged 1 commit into from
Oct 14, 2020

Conversation

yajo
Copy link
Contributor

@yajo yajo commented Oct 5, 2020

This feels more natural because multiline prompts usually start at the beginning of the terminal somehow. In a long question, the answer could start too far in the right side, and next lines too.

Of course, it can be overriden with the instruction parameter of the text prompt, so I'm just supplying a saner default.

This feels more natural because multiline prompts usually start at the beginning of the terminal somehow. In a long question, the answer could start too far in the right side, and next lines too.

Of course, it can be overriden with the `instruction` parameter of the `text` prompt, so I'm just supplying a saner default.
@kiancross kiancross requested a review from tmbo October 5, 2020 19:16
@kiancross kiancross added the Enhancement New feature or request label Oct 5, 2020
@yajo yajo mentioned this pull request Oct 6, 2020
6 tasks
Copy link
Owner

@tmbo tmbo left a comment

Choose a reason for hiding this comment

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

simple but great improvement to the UX, love it 🚀

@kiancross kiancross merged commit bbdfad5 into tmbo:master Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants