-
-
Notifications
You must be signed in to change notification settings - Fork 298
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
Eliminate vertical whitespace to better utilize available area #148
Comments
Hey, thanks for all the detailed suggestions. I'm happy you like our tool :) All of these suggested changes have trade-offs (like most things, I'd guess). It's important to me to illustrate those trade-offs here, and maybe hear your thoughts about them (or of course let anyone else chime in).
I will admit that I'm also not 100% fond of this line of space. This is the default for the terminal-ui library we're using, but I don't mind seeing it changed.
The reason we decided to put these in two separate lines is that currently the "Total Rate" display has no fixed width. Eg.
Wouldn't the separation become much less clear though? If we have full lines on all tables, it will be hard to tell where one ends and one begins? Or did I misunderstand? Aside from these issues: |
Ah sorry I thought this is an issue #305 fixes as well. My bad. |
Introduction: The following are some suggestions concerning the on-screen presentation that I'd like for you to consider. I respect the fact that you are the author of this application, and that you'll make the decisions you deem best.
1] Remove / re-purpose lines which are almost entirely blank.
I currently have an 80x26 window open and it is showing my Utilization by process name.
Below the line that says: "Process [ ... ] Connections [ ... ] Rate Up / Down" is a blank line.
You've already done a great job of using colors to differentiate the column headers from the data.
Considering removing the blank line by moving the lines of data up by one row. This will allow an additional row to be displayed on-screen.
2] Consolidate multiple lines where possible.
At the top of the screen I see "Total Rate Up / Down: 25.93MBps / 329.44KBps". At the bottom of the screen I see "Press to pause." I'd like to suggest that these two elements could be combined into a single line either at the top of the screen or the bottom (no preference here).
3] Connect the boxes when all three windows are displayed
I don't know if this will have a negative impact on the visual aesthetics and clarity (or if it will be neutral) but it seems that when you have three windows on screen (top left, top right, bottom), you could actually merge/connect the windows at the middle of the screen and free up another vertical line of real-estate to be repurposed.
Closing: Let me know if I should attach pictures to better illustrate this request. (I believe you, as the author, are probably familiar with what I'm saying here, but it may not be obvious to others who browse upon this issue. Again, please do what works best for you. This is distinct from the topic we discussed a few days ago, and I will be submitting a well-defined Issue which best proposes the enhancement.
PS: Very nice utility! I think your clever use of wide & tall terminal windows really sets you apart from so many other termimal-based applications.
The text was updated successfully, but these errors were encountered: