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

UI Zoom: multiple issues #6212

Closed
DorpsGek opened this issue Jan 17, 2015 · 6 comments
Closed

UI Zoom: multiple issues #6212

DorpsGek opened this issue Jan 17, 2015 · 6 comments
Labels
enhancement Issue would be a good enhancement; we accept Pull Requests! flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay

Comments

@DorpsGek
Copy link
Member

frosch opened the ticket and wrote:

Samu's problems with attached screenshots:
* Chat too small.
* News: "Manager" cut off, too small
* Statusbar too small.

Possibly scale default sizes of all windows. (unless custom size stored)

Attachments

Reported version: Version?
Operating system: All


This issue was imported from FlySpray: https://bugs.openttd.org/task/6212
@DorpsGek
Copy link
Member Author

ST2 wrote:

noticed that the screenshots were taken from our 1.5.0-beta1 test server (pompously called server # XX) - actually serves to test beta1 and some of GS's we use (this particular one is based on Aphid's CB gamescript).
Until now, server side, all goes without problems, and hopefully it's not a server setting that causes this. But if anything we can help, BTPro team is available :)


This comment was imported from FlySpray: https://bugs.openttd.org/task/6212#comment13725

@DorpsGek
Copy link
Member Author

DorpsGek commented Feb 3, 2015

frosch wrote:

Experimental implementation attached. At quad-zoom everything is so huge now, that it is unusable on a normal 1920x1080 screen. But maybe that is correct :)

The reason why this is so complicated, is that windows need to resize when changing the GUI zoom setting. As such the window must remember the original size restrictions, and need to distinguish them by unit (real pixels, traditional pixels, text lines).

Some places do widget size computation in the constructor. Likely these need to be rewritten to properly use UpdateWidgetSize.

The chat width thingie is a completely different issue. Currently there is a setting "gui.network_chat_box_width" which uses pixels and defaults to 620. Currently I think about replacing this "pixel setting" is a "percentage setting", so that it scales with window size.

Attachments


This comment was imported from FlySpray: https://bugs.openttd.org/task/6212#comment13757

@DorpsGek
Copy link
Member Author

frosch wrote:

Chat, statusbar and some windows have been fixed.

Highscore and end-game are mostly broken.

The smallest-size computations from above patch remain valid.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6212#comment13775

@DorpsGek
Copy link
Member Author

DorpsGek commented Aug 6, 2017

andythenorth wrote:

Station list vehicle-type icons are borked (truncated) :)


This comment was imported from FlySpray: https://bugs.openttd.org/task/6212#comment14498

@DorpsGek
Copy link
Member Author

andythenorth wrote:

See also #6266 - related but different.


This comment was imported from FlySpray: https://bugs.openttd.org/task/6212#comment14619

@DorpsGek DorpsGek added Core flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) labels Apr 7, 2018
@TrueBrain TrueBrain added patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay enhancement Issue would be a good enhancement; we accept Pull Requests! and removed bug from FlySpray labels Apr 13, 2018
@frosch123 frosch123 removed the Core label Apr 14, 2018
@andythenorth
Copy link
Contributor

Thanks for this. There's been no activity on this for some time, and as it stands, it doesn't look likely that it will go any further. I'm closing it as we try to keep the issue count low for OpenTTD, it helps us focus on things that are important and fun. Feel free to discuss in irc or request re-opening if you disagree. Thanks for contributing!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue would be a good enhancement; we accept Pull Requests! flyspray This issue is imported from FlySpray (https://bugs.openttd.org/) patch from FlySpray This issue is in fact a Patch, but imported from FlySrpay
Projects
None yet
Development

No branches or pull requests

4 participants