Skip to content
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

Add scroll bars to the city screen when info doesn't fit #1607

Merged
merged 1 commit into from
Dec 21, 2022

Conversation

lmoureaux
Copy link
Contributor

The layout of the city dialog was getting bad on 1366x768 screens because there was not enough vertical space. Add scroll bars as a minimal change for 3.0.

Closes #1584.

Need to compile twice since this changes the .ui

The layout of the city dialog was getting bad on 1366x768 screens because there
was not enough vertical space. Add scroll bars as a minimal change for 3.0.

Closes longturn#1584.
@lmoureaux lmoureaux requested a review from jwrober December 16, 2022 19:59
@jwrober
Copy link
Collaborator

jwrober commented Dec 16, 2022

At 720p. Any way to move this side of the box over to the left?
image

@lmoureaux
Copy link
Contributor Author

This is strange, I'll investigate

@lmoureaux
Copy link
Contributor Author

I couldn't reproduce, anything special I need to know? (Is this LTT/Royale?)

@jwrober
Copy link
Collaborator

jwrober commented Dec 21, 2022

I couldn't reproduce, anything special I need to know? (Is this LTT/Royale?)

It's a Royale game. Here is the save used to take the pic.

freeciv21-T0041-Y-2000-manual.sav.gz

@lmoureaux
Copy link
Contributor Author

I'm getting the same

@lmoureaux
Copy link
Contributor Author

The layout is strange, has a separator on the left but not on the right and nothing is centered:
image
Would suggest to let it through, it's not a regression.

@jwrober
Copy link
Collaborator

jwrober commented Dec 21, 2022

We've got this issue #728 that talks about centering the dialog better.

@jwrober jwrober merged commit 299a820 into longturn:master Dec 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The city dialog needs too much vertical space
2 participants