neix icon indicating copy to clipboard operation
neix copied to clipboard

[FEATURE] Optimize for termux

Open DhruvaSambrani opened this issue 4 years ago • 4 comments

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 avatar Aug 26 '21 12:08 DhruvaSambrani

#18 has some thing on keybindings. Should that be implemented first? Then termux keybindings will work as usual

DhruvaSambrani avatar Aug 26 '21 14:08 DhruvaSambrani

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 avatar Aug 27 '21 08:08 DhruvaSambrani

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

DhruvaSambrani avatar Sep 02 '21 20:09 DhruvaSambrani

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

DhruvaSambrani avatar Sep 02 '21 20:09 DhruvaSambrani