19 Feb
2025
19 Feb
'25
12:37 p.m.
Rémi Bernon (@rbernon) commented about dlls/winex11.drv/window.c:
HRGN win_rgn; POINT pos;
- if (!data->managed && is_window_managed( data->hwnd, SWP_NOACTIVATE, &data->rects.window )) + if (!data->managed && is_window_managed( data->hwnd, SWP_NOACTIVATE, &data->rects.visible ))
Looks like this isn't correct either: `is_window_managed` calls `NtUserGetMonitorInfo` which returns the monitor rect in thread DPI (which can be anything but raw DPI), and compares against driver rects which are in raw DPI. I initially had moved `is_window_managed` to win32u, to avoid that problem but it ended up not being accepted and I forgot to fix it differently. -- https://gitlab.winehq.org/wine/wine/-/merge_requests/7362#note_95163