Skip to content

NMSh's own lines wrap at words in narrow windows - #349

Open
raiseCatError wants to merge 1 commit into
release/v0.18.0from
polish/notice-word-wrap
Open

raiseCatError wants to merge 1 commit into
release/v0.18.0from
polish/notice-word-wrap

Conversation

@raiseCatError

Copy link
Copy Markdown
Owner

Small UI polish: NMSh's own lines (/copy confirmations, notices, completion rows) wrapped mid-word in narrow windows (… · 5 l / ines). They now break between words, with continuation rows indented under the text after a leading glyph. A word wider than the row still breaks by character.

❯ /copy
  ⎿ Copied to clipboard · 60 characters · 3
    lines
  • Copying stays exact: a selection copies the stored line, not the added indent.
  • Shell output is unchanged: it keeps the terminal's column wrapping.
  • Test changes: three live tests that read the screen as joined rows now allow whitespace between words in the messages they match.

Base: release/v0.18.0 (#345). Merge #345 first. It is independent of the other PRs.

Verified with npm run verify and a 46-column capture of real NMSh.

/copy confirmations, notices and completion rows split mid-word at narrow
widths ('5 l' / 'ines'). Lines NMSh writes itself now break after a space,
with continuation rows hanging under the text after a leading glyph; a word
wider than the row still breaks by character. Each continuation row's
indent covers only added padding, so a selection copies the line exactly.
Shell output keeps the terminal's column wrapping.

Live tests that read joined screen rows match these messages across a
word break.

This branch has not been deployed

No deployments
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