Stop words breaking across lines.

[Kingsy]

Hello,

By default if you echo a line longer than the terminal if it happens land in the middle of the word the terminal does not carry the word over to the next line but splits it so some chars are on one line and others on the next. This makes things quite hard to read for a paragraph of text.

Is there a way of preventing this?

Thanks

[Kingsy]

Oh, I have also just realised that when using less it doesn't word wrap at all, any idea on that one too?
Its actually really strange, it trims the line depending on the width of the page, so if you inspect the you'll see the words on that line which stops at the width of the page, if you decrease the width and inspect again there are less words in the span directly relating to how many that were trimmed off due to the loss of width.

I am guessing this is just a setting I have missed?

[jcubic]

You can make echo wrap on words there is option for that in echo keepWords

less don't wrap because this how native less works you need to use left/right arrow to show more.

[Kingsy]

Ah thats good news about echo.

Ah you are absolutely right about less, but its unfortunate in my case as I want to use it to read paragraphs of text. Is there an easy way of turning off this functionality? so it word wraps like echo within the less command?

If not I will have to write a hack before it to insert \n chars in the right places, which I don't really want to do.

[jcubic]

You would need to modify less plugin. I can check how hard it would be to add options to less. You can also try it yourself.

[Kingsy]

That would be great if you could have a look.
I'll take a look at the file myself too as you say to investigate.

[jcubic]

You need to modify function print()
https://github.com/jcubic/jquery.terminal/blob/master/js/less.js#L164
You will need to use $.terminal.split_equal before the lines are splitted, so you can proper number of lines.
I can add later today the options: wrap and keepWords.

[jcubic]

or maybe even before the lines get printed so you don't split lines each time less render text that can happen a lot on scroll.

[jcubic]

I think it would be better to modify function run() here:
https://github.com/jcubic/jquery.terminal/blob/master/js/less.js#L238
that handle initial input and return lines. Note that input can be Array of strings or string and function. There are a bit of logic there is will be more complicated when new options will be added.

[Kingsy]

Hm, that run function isnt clear how it would take the fix.
I can see that just making var should_substring = false; does the job, but the starting vertical position of less is off if I just do that, so perhaps it needs something else.

Oh right. if I am understand what you said above correctly? should I wait and you are going to add options into less to do this today?

[jcubic]

Updated less plugin, it now have two more options: wrap and keepWords.

[Kingsy]

Ahh this is great.
Will this change automatically be available via npm?

[Kingsy]

Looks like it has the latest release on there. Sorry that was obvious.
Do you think you could create a new release so I can include the new flags via npm please?

[jcubic]

No, I need to release new version first, it make take few days I have on feature that I can add to this version, which will be 2.21.0.
Also I found one bug in parsing commands that I would like to fix as soon as possible.

[Kingsy]

Thanks!
I will keep an eye out for the new version. much appreciated.

[jcubic]

The version of the file is devel branch I've accidentally pushed to master so I've removed the commit that was adding the fix.

[Kingsy]

ok np. I'll wait for the new version instead of copying the js file and linking to that.

[Kingsy]

Hi, any word on getting this in a new release?

[jcubic]

Sorry I didn't had much time to fix the other issue to release new version. In fact I've completely forgot that I need to release new version. Will fix that other issue and create that other feature maybe in this week.
If you really need this I can release kind of weird beta version for 2.21.0-beta.1 you will be able to install it from npm using:

npm install jquery.terminal@beta

[jcubic]

I've just finished fixing one bug and added the feature that I wanted in 2.21.0 it will be released soon after GitHub solve their issues, because I want my CI script to run, so I it can test the code on the server. It's works locally but I want my tests to run on the server as well.

[jcubic]

Version 2.21.0 got released.

[king-c]

Thanks for this! I'll try out the new features today. Appreciate it.