-
Notifications
You must be signed in to change notification settings - Fork 440
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
GlfwApplication: Workaround for DPI scaling #388
base: master
Are you sure you want to change the base?
Conversation
Merged as 411e349, thank you! |
@costashatz looking at release notes of glfw 3.3.1, I see "zero monitor size" being fixed: https://www.glfw.org/changelog.html .. should we remove this workaround again? (I assume distros that already have 3.3 would get 3.3.1 soon, and those that are on 3.2 and older won't upgrade to 3.3.0 because that would make no sense.) |
Codecov Report
@@ Coverage Diff @@
## master #388 +/- ##
==========================================
+ Coverage 72.4% 74.07% +1.67%
==========================================
Files 354 364 +10
Lines 18753 21099 +2346
==========================================
+ Hits 13578 15630 +2052
- Misses 5175 5469 +294
Continue to review full report at Codecov.
|
@mosra sorry for the late reply! I was on vacation.. I will check whether on my system this was fixed. Because this was a singularity container and maybe the version freezes to 3.3.. Does it hurt to keep it there? I mean performance-wise. |
No worries :) Performance-wise it doesn't hurt, but it's a no-longer-needed cruft that enlarges the surface for potential bugs. If you can confirm this is no longer needed, I'll remove it, otherwise I'll at least add a note that this workaround is obsolete since 3.3.1. |
This PR offers a workaround for DPI scaling when monitor size is equal to zero in GlfwApplication..