added textfill commandline argument

[davidbramsay]

I wrote this command line tool to center one line of text on the screen and make it as large as possible for the screen size. It takes one argument - the string to print - and centers/sizes it appropriately.

sudo papirus-textfill "hello world"

Handles up to 40 chars well, if the input is larger than 40 it will truncate it to 40.

[shawaj]

Looks great. Is there a restriction on line length and what happens if you write a large amount of text? Does it wrap to next lines?

Just want usage notes to add to readme

[davidbramsay]

hey @shawaj! It won't wrap, it simply truncates anything longer than 40 chars to the first 40 and prints it all on one line (centered, with as large a fontsize as will still fit on the screen).

[shawaj]

Awesome. Can you do a pull request adding that info to the main readme at the bottom any we will accept both?

On 24 Jan 2017 8:43 pm, "David Ramsay" notifications@github.com wrote:

hey @shawaj https://github.com/shawaj! It won't wrap, it simply truncates anything longer than 40 chars to the first 40 and prints it all on one line (centered, with as large a fontsize as will still fit on the screen).

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PiSupply/PaPiRus/pull/70#issuecomment-274932504, or mute the thread https://github.com/notifications/unsubscribe-auth/ADNCumcnx4rcO9CqkK1ii0_uBsQC0RL6ks5rVmJugaJpZM4LsoGp .

[davidbramsay]

I added it to the bottom of the command line section-- happy to put it under demos instead if that makes more sense (wasn't sure). Let me know if you have other comments!

[shawaj]

I think that'll be fine. We need to tidy up and overhaul the readme anyway at some point because it's a bit of a mess and some of our outdated.

As long as it's there somewhere for now that's great :-)

On 25 Jan 2017 4:31 am, "David Ramsay" notifications@github.com wrote:

I added it to the bottom of the command line section-- happy to put it under demos instead if that makes more sense (wasn't sure). Let me know if you have other comments!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/PiSupply/PaPiRus/pull/70#issuecomment-275016545, or mute the thread https://github.com/notifications/unsubscribe-auth/ADNCujIvKOXgyMQQgzm5rJJfXl5j09vYks5rVtAJgaJpZM4LsoGp .

[davidbramsay]

cool! I think it should be ready to merge :)

[francesco-vannini]

Checked and tested. Resolved a conflict with the README.md file, we'll also need to change the script to root check as in #79. Will mark as issue and solve straight away.

[francesco-vannini]

Thanks @dramsay9