[improvement] text wrapping improvements

[cylewitruk]

We should look into a way to better wrap text automatically based on current terminal width, ideally for all components. Here's an example of a note which has a long mnemonic and turns into chaos :)

Not a blocker, I'll wrap this manually for now, but I think it's worth discussing as it's one less thing for users to think about if we can find a sensible default solution :)

P.S. I'm currently using textwrap for this manually.

[fadeevab]

@cylewitruk At this point, I guess you just hate having 0 issues in this repo 😄 JK. I will think about what could be done with this.

[cylewitruk]

@cylewitruk At this point, I guess you just hate having 0 issues in this repo 😄 JK. I will think about what could be done with this.

@fadeevab haha, or rather, I was waiting for the issue count to go down before adding more 🤣

Don't feel like you need to address everything yourself, I'm happy to do some legwork as well :) But worth discussing anyway

[fadeevab]

@cylewitruk Basically, it seems like that textwrap should be inserted properly in one place.

For notes, it's format_note_generic with a terminal width argument (because the Term::stderr can be accessed in the lib.rs but must not be in the theme.rs). Moreover, textwrap is already a dependency for cliclack...

[cylewitruk]

@fadeevab I wonder if this could be made even more central, with a "padding" argument to account for things like a note which has prefix/suffix (effectively reducing width for wrapping by the provided padding)? The log methods already have this functionality, and if we could consolidate that in a way that all components can use it, that could be beneficial?

[corysabol]

Has there been any movement on this?
I'm very interested in proper text wrapping in the note() output in particular.

I'm happy to also take a stab at implementing this myself if the help is needed / work on it has stalled as I'm already digging around in the project to implement a different feature I would like.

[fadeevab]

@corysabol Hey! No movements, but PRs are appreciated!

[corysabol]

@fadeevab cool thanks for the update. I'm happy to try and get this improved and PR it back. Cheers!