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 scale support to window.py #76

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add scale support to window.py #76

wants to merge 1 commit into from

Conversation

AndrejGobeX
Copy link
Member

Added fix for scaled desktops. When Windows is scaled in the display settings, the script won't detect the edges of the Trackmania window properly. This fix makes the script makes the process DPI aware and can work under scaled environments. Tested on Windows 10 (however additional checks are encouraged).

Added fix for scaled desktops. When Windows is scaled in the display settings, the script won't detect the edges of the Trackmania window properly. This fix makes the script makes the process DPI aware and can work under scaled environments.
@yannbouteiller yannbouteiller self-assigned this Dec 1, 2023
yannbouteiller
yannbouteiller previously approved these changes Dec 1, 2023
@yannbouteiller yannbouteiller dismissed their stale review December 1, 2023 23:42

Doesn't work on Windows 11

@yannbouteiller
Copy link
Member

yannbouteiller commented Dec 1, 2023

Tested in Windows 11 with 150% scaling on my primary monitor, but this didn't work sadly.

(tested with python -m tmrl --check-environment: the captured portion of the screen did not match the window)

@yannbouteiller
Copy link
Member

Any news for this PR? It would be a nice-to-have but didn't work in my tests.

@AndrejGobeX
Copy link
Member Author

Hey, recently got a job and didn't have time to work on this... I might give it a spin someday.

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.

2 participants