In contrast to less: mdless doesn't adapt to changing view port of terminal

[leder11011]

As seen in the two screenshots less uses the full width of the terminal when the viewport is switched from portrait to landscape and mdless keeps the portrait formatting.

Thanks for a great app!

Unfortunately I cannot attach screenshots in GitHub mobile app.

[leder11011]

Could you please fix this issue for gem package? It's annoying and doesn't offer "a replacement for less".

[ttscoff]

mdless calculates its line wrapping and code formatting based on the width of the screen at the time the command is run. There is no way to output what it does in a way that will wrap if the screen width is changed.

[leder11011]

Hello Brett, Thank you for your answer and your commitment! It's exactly what less does: it respects view port changes of the terminal on the fly. Please reconsider! Bye Gerrit P. S. Screenshots attached Brett Terpstra ***@***.***> schrieb am Fr., 15. Sept. 2023, 14:39:

…

mdless calculates its line wrapping and code formatting based on the width of the screen at the time the command is run. There is no way to output what it does in a way that will wrap if the screen width is changed. — Reply to this email directly, view it on GitHub <#88 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAZNARBO524YFJODCOTALU3X2REAFANCNFSM6AAAAAA3KDRVKI> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[leder11011]

[leder11011]

Bump @ttscoff

[leder11011]

thank you for version 2.1.13!

Unfortunately I get a runtime error on Android termux, which was working fine on the previous version 1.0.37

Error is as follows:
console.rb:778:in '*': negative argument (Argument Error)

Please let me know if you need a stacktrace - I can send a screenshot, copy&pasta is a bit difficult on Android smartphone...

[leder11011]

https://photos.app.goo.gl/82WxiKzHutq3KQdcA

[ttscoff]

I believe this is fixed (current version 2.1.17).

…

On 4 Dec 2023, at 9:20, leder11011 wrote: thank you for version 2.1.13! Unfortunately I get a runtime error on Android termux, which was working fine on the previous version 1.0.37 Error is as follows: console.rb:778:in '*': negative argument (Argument Error) Please let me know if you need a stacktrace - I can send a screenshot, copy&pasta is a bit difficult on Android smartphone... -- Reply to this email directly or view it on GitHub: #88 (comment) You are receiving this because you were mentioned. Message ID: ***@***.***>

[leder11011]

Thank you for version 2.1.23

I have a regression now:

Opening mdless already in landscape mode uses the landscape width only partly, cp. Screenshot

[ttscoff]

Is there a width setting hardcoded in ~/.config/mdless/config.yml? It may have stored a current column width and then defaulted to it. If that’s the case I should remove that from automatic saving, so let me know. Thanks, Brett

…

On Dec 4, 2023 at 1:04 PM -0600, leder11011 ***@***.***>, wrote: Thank you for version 2.1.23 I have a regression now: Opening mdless already in landscape mode uses the landscape width only partly, cp. Screenshot Screenshot_20231204-195932.png (view on web) — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: ***@***.***>

[leder11011]

Hello Brett,

deleting the line :width: from config.yml did the trick! Please remove it from auto saving, too.

Best regards
Gerrit

[ttscoff]

I will delete, but it will mean any time a user wants to set a permanent width setting they'll have to manually include the :width: key, and running --update-theme will delete it. I'll just document it well.

…

-Brett

On 5 Dec 2023, at 1:50, leder11011 wrote: Hello Brett, deleting the line `:width:` from `config.yml` did the trick! Please remove it from auto saving, too. Best regards Gerrit -- Reply to this email directly or view it on GitHub: #88 (comment) You are receiving this because you were mentioned. Message ID: ***@***.***>

[leder11011]

Thank you!

2.1.25 is breaking with runtime error

[ttscoff]

Try 2.1.26. If that doesn't fix it, please attach a sample file that can replicate the error for you.

…

-Brett

On 5 Dec 2023, at 7:21, leder11011 wrote: Thank you! 2.1.25 is breaking with runtime error  -- Reply to this email directly or view it on GitHub: #88 (comment) You are receiving this because you were mentioned. Message ID: ***@***.***>