-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TUI in small windows wastes 50% of screen #137
Comments
Thank you for the feedback! PR #138 will improve this by removing the spacing for the debug header and the space for an expanded help panel. Check out this screenshot: Do you think that this is sufficient? I could also look into make it even more compact, but I'm inclined to start with this improvement rather than making the TUI overly dense. |
If you run |
Thank you for the quick update! Also, that update command is great! I updated to:
Top spacingIts better in that i get that first history entry now instead of none. However, these two lines seem to be wasted space as once they disappear, its just a blank line (or two sometimes depending on load time of the remote):
It's not ideal that the space used by those messages isn't always reclaimed. Or if its not meant to be reclaimed, i'd love for those messages to only be shown under a debug or high verbosity mode. Bottoms spacingThis update also seems to introduce a weird spacing on the bottom margin: Seems like if that bottom margin were cleaned up we could get maybe 3 or so history entries where we currently have 1. When combined with the top margin mentioned above, we could probably get 5 or 6 total entries in the list even in a very small terminal like I'm using in idea. |
For reference, the window size im using is:
|
Got it, thank you for the explanation and details! I've pushed out another update with improved handling for these extra small terminals (see #140 for the code). Please give it a shot and let me know what you think. |
Looks great! Now the smallest window i can do before showing no entries is 7 rows which is way too small to be useful anyway. Thanks for the great work! lmk if i can buy you a coffee. |
When using hishtory in a small window like those within an IDE, the whitespacing causes for a lot of wasted screen usage and when small enough, it will report that there are no history entries.
TUI erroneously showing no entries:
TUI showing entries but wasting half of the usable vertical space:
Is it possible to have a config option for more compact screen usage? There are a few approaches i could see:
It looks like the space between "Search Query" and the prompt is used for debug messages, but those don't seem to be super useful compared to the history information being sought. It would be nice to either have those debug messages in the bottom footer, or make it so when they disappear the space being used is taken back to allow for history result to have the entire screen.
The footer might be useful in the first few invocations, but when using the tool a lot its just redundant information that takes space from history items.
The spacing between history items and the "Search Query" line could be reduced.
The column headers seem to have a lot of spacing around them.
Any of these changes would be useful for smaller windows, but in general, my opinion is that the history items should be the first priority in terms of screen real estate, with everything else being optional.
If this seems like something that would be useful I can look into making a patch.
Thanks!
The text was updated successfully, but these errors were encountered: