qw3rtty / neix

neix - a RSS/Atom feed reader for your terminal.

GNU General Public License v3.0

200 stars 12 forks source link

[FEATURE] Optimize for termux #23

Open DhruvaSambrani opened 2 years ago

DhruvaSambrani commented 2 years ago

Starting a discussion to optimize for termux

The following are my opinions

UI Changes

[x] Reorder layout to have feeds tab above the article tab, and display only current feed name
[x] Do not display time and reduce whitespace in feed title

Behavior changes

[ ] Add --minimal command line option to activate "termux mode"
[x] ~~Add tab and shift-tab to change feed~~ Add PGUP and PGDN to change feed
[x] Reload screen size on terminal resize (Needed as keyboard can be closed after running neix)

DhruvaSambrani commented 2 years ago

18 has some thing on keybindings. Should that be implemented first?

Then termux keybindings will work as usual

DhruvaSambrani commented 2 years ago

Note that resizing is already supposed to work, but requires an extra keypress post resizing. This is a quirk of the ncurses library, and we can't do much.

Further, presently the resize is handled by KEY_RESIZE, but according to an SO post, not all ncurses are compiled with that enabled. In order to avoid this, we can either handle the signal ourselves, or, more simply, add a keybinding r to force resize.

DhruvaSambrani commented 2 years ago

PGUP and PGDN should change feed, not tab and shift-tab as those are the ones shown by default.

DhruvaSambrani commented 2 years ago

28 should complete all tasks except adding a command line configuration. Instead, it uses screen width as a heuristic for portrait orientation