Description
WrapLimit(0) is not treated as a limit — it silently behaves like WrapContent(true) (terminal-width detection), and on a non-TTY fails with:
cannot determine terminal width; use WrapLimit to set an explicit wrap limit when wrapping on non-TTY outputs
— advising the caller to do exactly what they just did.
Proposed behavior
Reject non-positive limits early with a clear message ("wrap limit must be positive; use WrapContent(true) for automatic terminal-based wrapping"), consistent with the existing negative-padding/margin validation. Keeps the API orthogonal: WrapLimit = explicit, WrapContent = auto. Also fixes the grammar of the multiline-title error ("only supported Inside title position only") in the same pass.
Fix prepared locally; will land referencing this issue.
Description
WrapLimit(0)is not treated as a limit — it silently behaves likeWrapContent(true)(terminal-width detection), and on a non-TTY fails with:— advising the caller to do exactly what they just did.
Proposed behavior
Reject non-positive limits early with a clear message ("wrap limit must be positive; use WrapContent(true) for automatic terminal-based wrapping"), consistent with the existing negative-padding/margin validation. Keeps the API orthogonal:
WrapLimit= explicit,WrapContent= auto. Also fixes the grammar of the multiline-title error ("only supported Inside title position only") in the same pass.Fix prepared locally; will land referencing this issue.