Skip to content

Limit prompt line to screen width#704

Merged
vinc merged 2 commits intotrunkfrom
feature/max-prompt-line
Dec 16, 2024
Merged

Limit prompt line to screen width#704
vinc merged 2 commits intotrunkfrom
feature/max-prompt-line

Conversation

@vinc
Copy link
Owner

@vinc vinc commented Nov 27, 2024

There's currently no multiline support in the prompt library (see #343 for a WIP) and a lot of weird things happen when a line goes past the screen width so the simplest fix for now is to prevent that.

prompt

@vinc
Copy link
Owner Author

vinc commented Nov 27, 2024

The fix handles line editing:

prompt2

@vinc
Copy link
Owner Author

vinc commented Nov 27, 2024

But it breaks down using file completion at the end of a long line:

prompt3

@vinc vinc marked this pull request as ready for review December 16, 2024 14:14
@vinc vinc merged commit 8b69722 into trunk Dec 16, 2024
@vinc vinc deleted the feature/max-prompt-line branch December 16, 2024 14:16
@vinc vinc mentioned this pull request Dec 21, 2024
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.

1 participant