The `s` binding is a bit confusing when border is disabled or the text is too short
ilyagr opened this issue · comments
The s binding does nothing when the border is disabled or if the window fits on one screen, which is a bit confusing.
Perhaps b could be a tri-state toggle: no border, border without scrollbar, border with scrollbar?
When the scrollbar is on, the whole "border" could be drawn thick as though it was a huge scrollbar (just to make it less confusing that these are two different states).
I'm also wondering if there's a good way to set up the UI so that the border only shows when either 1) the title or history are shown or 2) the scrollbar is needed.
This could be command-line option, but I'm not currently sure what exactly the b keybinding should do in that case (or s if it still exists). It doesn't work perfectly with my tri-state idea since b would do nothing in certain cases. Perhaps the keybinding can be documented as "toggle border and scrollbar (when applicable)", I'm unsure.
Come to think of it, maybe you are right.
Since I had created the METHOD separately, I had implemented the key bindings separately for the time being.
In the next version at 0.3.15, change the default behavior to a toggle that enables/disables the border and scrollbars as a set.