Ensuring AdjustWindowRectExForDpi uses consistent zoom#3029
Ensuring AdjustWindowRectExForDpi uses consistent zoom#3029
Conversation
Changed Control.adjustWindowRectEx to use getShellZoom() instead of shell.getZoom(). This ensures consistency with nativeZoom which is used by computeTrim and other layout logic, preventing the possibility of incorrect border sizing when shell.getZoom() and nativeZoom are out of sync
This should only be the case if you have disabled autoscaling for a control. And in that case it's necessary to take the proper shell zoom for those metrics as otherwise scaling will be wrong. So from my understanding, the current implementation is correct as is. Can you give an example where the current implementation does not behave as expected? |
#3020 might be related. |
HeikoKlare
left a comment
There was a problem hiding this comment.
As said, I don't see that this can make anything better but will just break autoscale disablement at the widget level. Without any reproduce of an issue that this solves and the fact it will actually break behavior, this cannot be merged.


Changed Control.adjustWindowRectEx to use getShellZoom() instead of shell.getZoom(). This ensures consistency with nativeZoom which is used by computeTrim and other layout logic, preventing the possibility of incorrect
border sizing when shell.getZoom() and nativeZoom are out of sync