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

[Bug]: Cargo flow legend line thickness becomes larger with UI scaling #10255

Closed
merni-ns opened this issue Dec 18, 2022 · 3 comments · Fixed by #10410
Closed

[Bug]: Cargo flow legend line thickness becomes larger with UI scaling #10255

merni-ns opened this issue Dec 18, 2022 · 3 comments · Fixed by #10410
Labels
enhancement Issue would be a good enhancement; we accept Pull Requests! good first issue Good for newcomers
Milestone

Comments

@merni-ns
Copy link
Contributor

merni-ns commented Dec 18, 2022

Version of OpenTTD

13.0-beta2 on Windows 11

Expected result

Either the thickness of the cargo flow legend's lines remains the same regardless of UI scaling, or there is a separate setting to control the thickness of those lines (as there is with graph lines). To me, the most important thing in scaling is to make UI elements (esp. text) readable since I have a small laptop screen with a high (1920x1080) resolution. I don't want non-text elements in the game world like the cargo flow lines to be enlarged.

Actual result

The thickness of the cargo flow lines increases to a large extent when UI scaling is applied.
The below screenshot shows on the left JGRPP 0.47.2 (based on OpenTTD 12.x) and on the right JGRPP 0.50.0 (using the UI scaling feature of OpenTTD 13). In both windows, 2x scaling is applied (on the left it is 2x interface scaling + 2x font scaling).
Screenshot 2022-12-18 135418

Steps to reproduce

  1. Increase the UI scaling on OpenTTD 13.0-beta2 to 2x or any larger setting.
  2. Open a save with some transport network, and enable the "Cargo Flow Legend"
  3. Observe that the lines are much bigger than those in 12.x versions with 2x scaling.
@PeterN
Copy link
Member

PeterN commented Dec 18, 2022

This is 'intentional', caused by #10064. I'm not sure that is correct though.

This is not actually caused by the new UI scaling changes, it was added before that.

@merni-ns
Copy link
Contributor Author

merni-ns commented Dec 19, 2022 via email

@PeterN
Copy link
Member

PeterN commented Dec 19, 2022

I think thinning them down from 3px to 2px (per colour, so 6px total) at 1x scale may be good enough. They are then 'only' 12px wide at 2x scale instead of 18px.

Saves yet more options.

@2TallTyler 2TallTyler added this to the 13.0 milestone Dec 20, 2022
@2TallTyler 2TallTyler added good first issue Good for newcomers enhancement Issue would be a good enhancement; we accept Pull Requests! labels Dec 20, 2022
merni-ns added a commit to merni-ns/OpenTTD that referenced this issue Jan 25, 2023
…aph GUI lines

From 3px to 2px (multiplied by UI scale).
merni-ns added a commit to merni-ns/OpenTTD that referenced this issue Jan 25, 2023
PeterN pushed a commit that referenced this issue Jan 25, 2023
michicc pushed a commit to michicc/OpenTTD that referenced this issue Jan 27, 2023
michicc pushed a commit to michicc/OpenTTD that referenced this issue Jan 28, 2023
michicc pushed a commit that referenced this issue Jan 28, 2023
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! good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants