GlfwApplication: Workaround for DPI scaling#388
GlfwApplication: Workaround for DPI scaling#388costashatz wants to merge 1 commit intomosra:masterfrom
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..