From e9999fa61ea0ffa602c69e10e54996b3a636a5e6 Mon Sep 17 00:00:00 2001
From: Yuhang Zhao <2546789017@qq.com>
Date: 周五, 09 5月 2025 16:03:45 +0800
Subject: [PATCH] fix window reopen issue (#179)
---
src/core/contexts/win32windowcontext.cpp | 134 +++++++++++++++++++++++++++++++++++---------
1 files changed, 105 insertions(+), 29 deletions(-)
diff --git a/src/core/contexts/win32windowcontext.cpp b/src/core/contexts/win32windowcontext.cpp
index ce1037a..e155d3d 100644
--- a/src/core/contexts/win32windowcontext.cpp
+++ b/src/core/contexts/win32windowcontext.cpp
@@ -33,6 +33,18 @@
#include "qwkglobal_p.h"
#include "qwkwindowsextra_p.h"
+#if (QT_VERSION >= QT_VERSION_CHECK(6, 6, 0)) && (QT_VERSION <= QT_VERSION_CHECK(6, 6, 1))
+# error Current Qt version has a critical bug which will break QWK functionality. Please upgrade to > 6.6.1 or downgrade to < 6.6.0
+#endif
+
+#ifndef DWM_BB_ENABLE
+# define DWM_BB_ENABLE 0x00000001
+#endif
+
+#ifndef ABM_GETAUTOHIDEBAREX
+# define ABM_GETAUTOHIDEBAREX 0x0000000b
+#endif
+
namespace QWK {
enum IconButtonClickLevelFlag {
@@ -82,7 +94,10 @@
static void setInternalWindowFrameMargins(QWindow *window, const QMargins &margins) {
const QVariant marginsVar = QVariant::fromValue(margins);
- // TODO: Add comments
+ // We need to tell Qt we have set a custom margin, because we are hiding
+ // the title bar by pretending the whole window is filled by client area,
+ // this however confuses Qt's internal logic. We need to do the following
+ // hack to let Qt consider the extra margin when changing window geometry.
window->setProperty("_q_windowsCustomMargins", marginsVar);
#if QT_VERSION < QT_VERSION_CHECK(6, 0, 0)
if (QPlatformWindow *platformWindow = window->handle()) {
@@ -193,8 +208,8 @@
[[maybe_unused]] const auto &cleaner =
qScopeGuard([windowThreadProcessId, currentThreadId]() {
- ::AttachThreadInput(windowThreadProcessId, currentThreadId, FALSE); //
- }); // TODO: Remove it
+ ::AttachThreadInput(windowThreadProcessId, currentThreadId, FALSE);
+ });
::BringWindowToTop(hwnd);
// Activate the window too. This will force us to the virtual desktop this
@@ -215,12 +230,20 @@
return true;
}
+ const auto windowStyles = ::GetWindowLongPtrW(hWnd, GWL_STYLE);
+ const bool allowMaximize = windowStyles & WS_MAXIMIZEBOX;
+ const bool allowMinimize = windowStyles & WS_MINIMIZEBOX;
+
const bool maxOrFull = isMaximized(hWnd) || isFullScreen(hWnd);
::EnableMenuItem(hMenu, SC_CLOSE, (MF_BYCOMMAND | MFS_ENABLED));
- ::EnableMenuItem(hMenu, SC_MAXIMIZE,
- (MF_BYCOMMAND | ((maxOrFull || fixedSize) ? MFS_DISABLED : MFS_ENABLED)));
- ::EnableMenuItem(hMenu, SC_RESTORE,
- (MF_BYCOMMAND | ((maxOrFull && !fixedSize) ? MFS_ENABLED : MFS_DISABLED)));
+ ::EnableMenuItem(
+ hMenu, SC_MAXIMIZE,
+ (MF_BYCOMMAND |
+ ((maxOrFull || fixedSize || !allowMaximize) ? MFS_DISABLED : MFS_ENABLED)));
+ ::EnableMenuItem(
+ hMenu, SC_RESTORE,
+ (MF_BYCOMMAND |
+ ((maxOrFull && !fixedSize && allowMaximize) ? MFS_ENABLED : MFS_DISABLED)));
// The first menu item should be selected by default if the menu is brought
// up by keyboard. I don't know how to pre-select a menu item but it seems
// highlight can do the job. However, there's an annoying issue if we do
@@ -232,7 +255,8 @@
// the menu look kind of weird. Currently I don't know how to fix this issue.
::HiliteMenuItem(hWnd, hMenu, SC_RESTORE,
(MF_BYCOMMAND | (selectFirstEntry ? MFS_HILITE : MFS_UNHILITE)));
- ::EnableMenuItem(hMenu, SC_MINIMIZE, (MF_BYCOMMAND | MFS_ENABLED));
+ ::EnableMenuItem(hMenu, SC_MINIMIZE,
+ (MF_BYCOMMAND | (allowMinimize ? MFS_ENABLED : MFS_DISABLED)));
::EnableMenuItem(hMenu, SC_SIZE,
(MF_BYCOMMAND | ((maxOrFull || fixedSize) ? MFS_DISABLED : MFS_ENABLED)));
::EnableMenuItem(hMenu, SC_MOVE, (MF_BYCOMMAND | (maxOrFull ? MFS_DISABLED : MFS_ENABLED)));
@@ -351,8 +375,8 @@
static MSG createMessageBlock(HWND hWnd, UINT message, WPARAM wParam, LPARAM lParam) {
MSG msg;
- msg.hwnd = hWnd; // re-create MSG structure
- msg.message = message; // time and pt fields ignored
+ msg.hwnd = hWnd;
+ msg.message = message;
msg.wParam = wParam;
msg.lParam = lParam;
@@ -362,6 +386,8 @@
if (!isNonClientMessage(message)) {
::ScreenToClient(hWnd, &msg.pt);
}
+
+ msg.time = ::GetMessageTime();
return msg;
}
@@ -396,7 +422,7 @@
}
// Send to QAbstractEventDispatcher
- bool filterNativeEvent(MSG *msg, LRESULT *result) {
+ static bool filterNativeEvent(MSG *msg, LRESULT *result) {
auto dispatcher = QAbstractEventDispatcher::instance();
QT_NATIVE_EVENT_RESULT_TYPE filterResult = *result;
if (dispatcher && dispatcher->filterNativeEvent(nativeEventType(), msg, &filterResult)) {
@@ -407,7 +433,7 @@
}
// Send to QWindowSystemInterface
- bool filterNativeEvent(QWindow *window, MSG *msg, LRESULT *result) {
+ static bool filterNativeEvent(QWindow *window, MSG *msg, LRESULT *result) {
QT_NATIVE_EVENT_RESULT_TYPE filterResult = *result;
if (QWindowSystemInterface::handleNativeEvent(window, nativeEventType(), msg,
&filterResult)) {
@@ -620,6 +646,11 @@
// Save window handle mapping
g_wndProcHash->insert(hWnd, ctx);
+
+ // Force a WM_NCCALCSIZE message manually to avoid the title bar become visible
+ // while Qt is re-creating the window (such as setWindowFlag(s) calls). It has
+ // been observed by our users.
+ triggerFrameChange(hWnd);
}
static inline void removeManagedWindow(HWND hWnd) {
@@ -684,7 +715,7 @@
}
#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDERS)
- case DrawWindows10BorderHook: {
+ case DrawWindows10BorderHook_Emulated: {
if (!m_windowId)
return;
@@ -727,7 +758,7 @@
return;
}
- case DrawWindows10BorderHook2: {
+ case DrawWindows10BorderHook_Native: {
if (!m_windowId)
return;
@@ -894,19 +925,11 @@
const auto hwnd = reinterpret_cast<HWND>(m_windowId);
Q_ASSERT(hwnd);
- if (!hwnd) {
- return false;
- }
const DynamicApis &apis = DynamicApis::instance();
const auto &extendMargins = [this, &apis, hwnd]() {
- // For some unknown reason, the window background is totally black when the host object
- // is a QWidget. And extending the window frame into the client area seems to fix it
- // magically.
- // We don't need the following *HACK* for QtQuick windows.
- if (!m_host->isWidgetType()) {
- return;
- }
+ // For some unknown reason, the window background is totally black and extending
+ // the window frame into the client area seems to fix it magically.
// After many times of trying, we found that the Acrylic/Mica/Mica Alt background
// only appears on the native Win32 window's background, so naturally we want to
// extend the window frame into the whole client area to be able to let the special
@@ -922,14 +945,34 @@
// from the top side. In the end I found that we only need to extend from the left
// side if we extend long enough. In this way we can see the special material even
// when the host object is a QWidget and the title bar still remain hidden. But even
- // though this solution seems perfect, I really don't know why it works.
+ // though this solution seems perfect, I really don't know why it works. The following
+ // hack is totally based on experiments.
static constexpr const MARGINS margins = {65536, 0, 0, 0};
apis.pDwmExtendFrameIntoClientArea(hwnd, &margins);
};
const auto &restoreMargins = [this, &apis, hwnd]() {
auto margins = qmargins2margins(
- m_windowAttributes.value(QStringLiteral("extra-margins")).value<QMargins>());
+ windowAttribute(QStringLiteral("extra-margins")).value<QMargins>());
apis.pDwmExtendFrameIntoClientArea(hwnd, &margins);
+ };
+
+ const auto &effectBugWorkaround = [this, hwnd]() {
+ // We don't need the following *HACK* for QWidget windows.
+ // Completely based on actual experiments, root reason is totally unknown.
+ if (m_host->isWidgetType()) {
+ return;
+ }
+ static QSet<WId> bugWindowSet{};
+ if (bugWindowSet.contains(m_windowId)) {
+ return;
+ }
+ bugWindowSet.insert(m_windowId);
+ RECT rect{};
+ ::GetWindowRect(hwnd, &rect);
+ ::MoveWindow(hwnd, rect.left, rect.top, 1, 1, FALSE);
+ ::MoveWindow(hwnd, rect.right - 1, rect.bottom - 1, 1, 1, FALSE);
+ ::MoveWindow(hwnd, rect.left, rect.top, rect.right - rect.left, rect.bottom - rect.top,
+ FALSE);
};
if (key == QStringLiteral("extra-margins")) {
@@ -948,7 +991,8 @@
} else {
apis.pAllowDarkModeForApp(enable);
}
- const auto attr = isWin1020H1OrGreater() ? _DWMWA_USE_IMMERSIVE_DARK_MODE : _DWMWA_USE_IMMERSIVE_DARK_MODE_BEFORE_20H1;
+ const auto attr = isWin1020H1OrGreater() ? _DWMWA_USE_IMMERSIVE_DARK_MODE
+ : _DWMWA_USE_IMMERSIVE_DARK_MODE_BEFORE_20H1;
apis.pDwmSetWindowAttribute(hwnd, attr, &enable, sizeof(enable));
apis.pFlushMenuThemes();
@@ -985,6 +1029,7 @@
}
restoreMargins();
}
+ effectBugWorkaround();
return true;
}
@@ -1005,6 +1050,7 @@
sizeof(backdropType));
restoreMargins();
}
+ effectBugWorkaround();
return true;
}
@@ -1050,6 +1096,7 @@
restoreMargins();
}
+ effectBugWorkaround();
return true;
}
@@ -1089,6 +1136,7 @@
apis.pDwmEnableBlurBehindWindow(hwnd, &bb);
}
}
+ effectBugWorkaround();
return true;
}
return false;
@@ -1566,7 +1614,7 @@
bool isInTitleBar = isInTitleBarDraggableArea(qtScenePos);
WindowAgentBase::SystemButton sysButtonType = WindowAgentBase::Unknown;
bool isInCaptionButtons = isInSystemButtons(qtScenePos, &sysButtonType);
- bool dontOverrideCursor = false; // ### TODO
+ static constexpr bool dontOverrideCursor = false; // ### TODO
if (isInCaptionButtons) {
// Firstly, we set the hit test result to a default value to be able to detect
@@ -1861,6 +1909,18 @@
break;
}
+ case WM_SHOWWINDOW: {
+ if (!wParam || !isWindowNoState(hWnd) || isFullScreen(hWnd)) {
+ break;
+ }
+ RECT windowRect{};
+ ::GetWindowRect(hWnd, &windowRect);
+ static constexpr const auto swpFlags = SWP_NOMOVE | SWP_NOZORDER | SWP_NOACTIVATE | SWP_FRAMECHANGED | SWP_NOOWNERZORDER;
+ ::SetWindowPos(hWnd, nullptr, 0, 0, RECT_WIDTH(windowRect) + 1, RECT_HEIGHT(windowRect) + 1, swpFlags);
+ ::SetWindowPos(hWnd, nullptr, 0, 0, RECT_WIDTH(windowRect), RECT_HEIGHT(windowRect), swpFlags);
+ break;
+ }
+
default:
break;
}
@@ -2014,8 +2074,24 @@
// implement an elaborate client-area preservation technique, and
// simply return 0, which means "preserve the entire old client area
// and align it with the upper-left corner of our new client area".
+
const auto clientRect = wParam ? &(reinterpret_cast<LPNCCALCSIZE_PARAMS>(lParam))->rgrc[0]
: reinterpret_cast<LPRECT>(lParam);
+ [[maybe_unused]] const auto &flickerReducer = qScopeGuard([this]() {
+ // When we receive this message, it means the window size has changed
+ // already, and it seems this message always come before any client
+ // area size notifications (eg. WM_WINDOWPOSCHANGED and WM_SIZE). Let
+ // D3D/VK paint immediately to let user see the latest result as soon
+ // as possible.
+ const auto &isTargetSurface = [](const QSurface::SurfaceType st) {
+ return st != QSurface::RasterSurface && st != QSurface::OpenGLSurface &&
+ st != QSurface::RasterGLSurface && st != QSurface::OpenVGSurface;
+ };
+ if (m_windowHandle && isTargetSurface(m_windowHandle->surfaceType()) &&
+ isDwmCompositionEnabled() && DynamicApis::instance().pDwmFlush) {
+ DynamicApis::instance().pDwmFlush();
+ }
+ });
if (isSystemBorderEnabled()) {
// Store the original top margin before the default window procedure applies the
// default frame.
@@ -2265,7 +2341,7 @@
if (!mouseHook) {
mouseHook = ::SetWindowsHookExW(
WH_MOUSE,
- [](int nCode, WPARAM wParam, LPARAM lParam) {
+ [](int nCode, WPARAM wParam, LPARAM lParam) -> LRESULT {
if (nCode >= 0) {
switch (wParam) {
case WM_LBUTTONDBLCLK:
--
Gitblit v1.9.1