-
Notifications
You must be signed in to change notification settings - Fork 42
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
Fix the units report when there are many units #2318
Conversation
Split the window horizontally instead of vertically, which facilitates scrolling. Closes longturn#2313.
I agree that your proposed layout looks better. Does it look ok when no units are waiting? |
The "units waiting" group is hidden in this case. Feel free to test my proposal (which is a good idea anyways): blabber/freeciv21@38f5dea |
@hugomflavio and I worked together to evaluate this PR as well. There are a few things we need to think about on this before its ready if we keep the change horizontally opposed:
All that said, I personally would prefer that units waiting is vertically opposed to units like I did it initially. It works best on smaller displays, which we have many players in this category. Setting some limiters so the units table can't take up the whole screen and bury the units waiting table when there are many units in both tables is important. My thinking is when units waiting is present we change the geometry of the units table so it is max half screen "tall" with a horizontal slider in the middle. Then we see the units waiting table at the bottom half, the default slider in the middle is at mid-screen and should allow the player to slide up/down to see more/less of each table on the screen as needed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see note above
Another alternative idea is to change the view to a tabbed interface with units table on one tab and units waiting on a second. Then each table can use the whole screen. |
A resizeable split screen can be done with a |
Split the window horizontally instead of vertically, which facilitates scrolling.
Closes #2313.