You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Edit: Let's not treat this as a bug. I don't want to mess with the --short behavior after all these years, in case people are reliant on it for parsing jrnl output. Instead, let's document which formats are supported and not supported by linewrap.
Diagnostic output
jrnl: v3.3
Python: 3.11.1 (tags/v3.11.1:a7a450f, Dec 6 2022, 19:58:39) [MSC v.1934 64 bit (AMD64)]
OS: Windows 10
Current Behavior
When running jrnl --short, the text isn't wrapped.
Expected Behavior
I would expect long titles to be wrapped with short because the config reference says the config value linewrap "controls the width of the output".
However... this behavior is probably also really old. If anyone is using \n to split jrnl --short output, they could have a bad time if we "fix" this for them. I'm open to compromises on this. For instance, we could close this issue as "won't fix" and document the current behavior better.
Edit: Let's not treat this as a bug. I don't want to mess with the
--short
behavior after all these years, in case people are reliant on it for parsing jrnl output. Instead, let's document which formats are supported and not supported bylinewrap
.Diagnostic output
jrnl: v3.3
Python: 3.11.1 (tags/v3.11.1:a7a450f, Dec 6 2022, 19:58:39) [MSC v.1934 64 bit (AMD64)]
OS: Windows 10
Current Behavior
When running
jrnl --short
, the text isn't wrapped.Expected Behavior
I would expect long titles to be wrapped with short because the config reference says the config value
linewrap
"controls the width of the output".However... this behavior is probably also really old. If anyone is using
\n
to splitjrnl --short
output, they could have a bad time if we "fix" this for them. I'm open to compromises on this. For instance, we could close this issue as "won't fix" and document the current behavior better.Repro Steps
Debug output
Other Information
Discovered while investigating #1191 (replace ansiwrap). The resolution to that issue could affect this one.
The text was updated successfully, but these errors were encountered: