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

Temperature is misaligned on wider screens #3

Open
ornellino opened this issue Jan 9, 2025 · 17 comments · Fixed by #7
Open

Temperature is misaligned on wider screens #3

ornellino opened this issue Jan 9, 2025 · 17 comments · Fixed by #7
Labels
bug Something isn't working staged
Milestone

Comments

@ornellino
Copy link

Hi, I have a very wide monitor (32/9, 5120x1440), so your mod misaligns the temperature on the left, above the weather icons.
There is a way to dinamically align based on monitor width?
Thanks in advance
Andrea

@Peppie84
Copy link
Owner

Peppie84 commented Jan 9, 2025

Hi @ornellino , thanks for your thread. Can you please add a screenshot that i can see the misalignment. thx

Fixed Version (2025-01-09)

FS25_ExtendedGameInfoDisplay.zip

@Peppie84 Peppie84 added the bug Something isn't working label Jan 9, 2025
@kayento
Copy link

kayento commented Jan 9, 2025

fsScreen_2025_01_09_17_28_22

Hello, I hope it's ok if I use this post. Thanks for the great mod. I have a screen resolution of 3440x1440 (21:9) and the same problem. Maybe this can be solved. Thank you very much.

@Peppie84
Copy link
Owner

Peppie84 commented Jan 9, 2025

Hi @kayento no problem. Thanks for your screen. Can you send me also a screen without the mod. Just for checking

@Shad0wlife
Copy link

This also happens on 21:9 Resolution (3440x1440):
grafik

This is from a save without the mod:
grafik

@kayento
Copy link

kayento commented Jan 9, 2025

Hi @kayento no problem. Thanks for your screen. Can you send me also a screen without the mod. Just for checking

Of Course:

fsScreen_2025_01_09_18_32_01

@Peppie84
Copy link
Owner

Peppie84 commented Jan 9, 2025

Thanks. I can reproduce the error by setting 2384 x 1024 (21:9) for me. I'll fix it.

@ornellino
Copy link
Author

Screenshot 2025-01-09 204838

Screenshot 2025-01-09 205130

Here we are

@Peppie84
Copy link
Owner

Peppie84 commented Jan 9, 2025

Someone here to re-try? It works now for my 21:9 test.
FS25_ExtendedGameInfoDisplay.zip

@archipiotr
Copy link

Now work fine for me. Tested in ultrawide 32:9 - 5120×1440 and wide 21:9 - 2560x1080 (samsung odyssey g9). Thanks!!!

32_9
21_9

@kayento
Copy link

kayento commented Jan 10, 2025

I can confirm that. It looks very good. Thank you.

@Peppie84
Copy link
Owner

Thanks for testing.

The problem was, that positioning of the temperature was calculated by the ScalePixelToHeight instead to use the ScalePixelToWidth for the X coordinate, for respecting the current used resolution.

Please don't miss to rate the mod, thanks 😅

Peppie84 added a commit that referenced this issue Jan 10, 2025
…ltra-widescreen-resolution

[#3] Fix temperature misalignment for widesceen resolutions
@Peppie84 Peppie84 reopened this Jan 10, 2025
@Peppie84 Peppie84 added this to the v1.0.0.1 milestone Jan 10, 2025
@ornellino
Copy link
Author

Screenshot 2025-01-10 095836
Also For me works fine! Thanks! Useful mod!

@NiDHUG85
Copy link

I just got your addon from farming-simulator.com and there is an issue with the resolution 2560x1080.
Is there an update? version on the website is V1.0.0.0
fsScreen_2025_01_14_10_40_23
fsScreen_2025_01_14_10_42_15

@Peppie84
Copy link
Owner

Hi @NiDHUG85 , there is currently a fix version for download: #3 (comment)

@NiDHUG85
Copy link

Hi @NiDHUG85 , there is currently a fix version for download: #3 (comment)

Thanks, it might be an idea to update the version on the website too :)

@Peppie84
Copy link
Owner

Hi @NiDHUG85 , there is currently a fix version for download: #3 (comment)

Thanks, it might be an idea to update the version on the website too :)

You mean Modhub? I only can deploy a new release after two weeks of a release.

@NiDHUG85
Copy link

Hi @NiDHUG85 , there is currently a fix version for download: #3 (comment)

Thanks, it might be an idea to update the version on the website too :)

You mean Modhub? I only can deploy a new release after two weeks of a release.

Ahhhh, I didn't knew that 😄
Just thought it would be a good idea to have it updated so people that get it would get the newest one 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working staged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants