qw3rtty / neix

neix - a RSS/Atom feed reader for your terminal.
GNU General Public License v3.0
200 stars 12 forks source link

Add $article_width variable for renderText option #24

Closed DhruvaSambrani closed 2 years ago

DhruvaSambrani commented 2 years ago

Add runtime var to send the actual width of the article to renderText command by text replacement

This allows renderText to have $article_width anywhere in the command which will be replaced by the width of the article display window.

Code Changes-

Other Changes-

Fixes #21

qw3rtty commented 2 years ago

@DhruvaSambrani Thanks for your work!

But there are some build errors when the Unit-Tests are enabled. Could you check that and fix?

DhruvaSambrani commented 2 years ago

I have fixed the error by changing param order and adding a default value. Should I also add a test case for this?

DhruvaSambrani commented 2 years ago

@tomschwarz please check

qw3rtty commented 2 years ago

@DhruvaSambrani Thanks for your fix! I really appreciate your work!