Skip to content
This repository has been archived by the owner on Apr 13, 2024. It is now read-only.

Improve text wrapping #75

Merged
merged 3 commits into from
Apr 10, 2024
Merged

Improve text wrapping #75

merged 3 commits into from
Apr 10, 2024

Conversation

AtkinsSJ
Copy link
Contributor

@AtkinsSJ AtkinsSJ commented Apr 4, 2024

  • Preserve newlines, and each input line's leading whitespace
  • Skip over ANSI escape sequences when measuring how wide the text is, so those lines don't wrap early

@AtkinsSJ
Preserve newlines and whitespace in wrapText()
af239d9 
Replaces the naive "split at whitespace and then recombine the parts
with ' '" approach with character-by-character iteration, preserving
the original newlines and line-start whitespace.
@AtkinsSJ
Ignore ANSI escape sequences when wrapping text
855d354 
These escape sequences don't take up any space, so we don't want to
include their characters when measuring how much will fit on a line.

For now, this only covers the graphics-mode sequences, since those are
the ones we actually use.
@AtkinsSJ
help: Wrap more of the command-usage text
5fc12d1
@KernelDeimos KernelDeimos merged commit 16ac295 into HeyPuter:trunk Apr 10, 2024
3 checks passed
@AtkinsSJ AtkinsSJ deleted the wrapping-whitespace branch April 10, 2024 14:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AtkinsSJ @KernelDeimos