Add dpiAwareness option to manifest and send physicial viewport size to Spout #3462
Conversation
hmartinez82
changed the title
hmartinez82
changed the title
|
Looking better :) |
hmartinez82
changed the title
|
Great PR! Please pay attention to the following items before merging: Files matching
This is an automatically generated QA checklist based on modified files. |
|
I tested moving the Stellarium window from a 200%DPI to a 125%DPI and a 100% DPI and back, the resize SIGNAL is also handling those cases. So, I don't have to manually handle the screenChanged events \o/ Left monitor 200% DPI, right monitor 125%DPI Main window in the 200% DPI monitor Main window in the 125% DPI monitor |
hmartinez82
force-pushed
the
PerMonitorV2
branch
from
89e9881
to
d2cc9c8
Compare
alex-w
added
the
state: published
|
Hello @hmartinez82! Please check the fresh version (development snapshot) of Stellarium: |
alex-w
removed
the
state: published
|
Hello @hmartinez82! Please check the latest stable version of Stellarium: |
Qt 6 defaults to PerMonitorV2 as its DPI awareness options.
Description
The ACCESS_DENIED error was happening because the setting was being set by the manifest file and then, when the QGuiApplication instance is being created, it calls
SetProcessDpiAwarenessContext()which clearly states that setting that after the manifest also sets it will lead to ACCESS_DENIED.Adding the options before Qt tries to set it ensures it won't try to call SetProcessDpiAwarenessContext() because the process will already be PerMonitorV2. (Qt checks the current options before trying to change it, see https://github.com/qt/qtbase/blob/v6.5.3/src/plugins/platforms/windows/qwindowscontext.cpp#L397)
Fixes #3461
Type of change