sola/qwindowkit.git

			@@ -117,7 +117,7 @@
			private:
			DynamicApis() {
			#define DYNAMIC_API_RESOLVE(DLL, NAME) \
			p##NAME = reinterpret_cast<decltype(p##NAME)>(DLL.resolve(#NAME))
			p##NAME = reinterpret_cast<decltype(p##NAME)>(DLL.resolve(#NAME))

			QSystemLibrary user32(QStringLiteral("user32"));
			DYNAMIC_API_RESOLVE(user32, GetDpiForWindow);
			@@ -480,21 +480,10 @@
			// so we use this small technique to pretend the foreground window is ours.
			::AttachThreadInput(windowThreadProcessId, currentThreadId, TRUE);

			struct Cleaner {
			Cleaner(DWORD idAttach, DWORD idAttachTo)
			: m_idAttach(idAttach), m_idAttachTo(idAttachTo) {
			}
			~Cleaner() {
			::AttachThreadInput(m_idAttach, m_idAttachTo, FALSE);
			}

			private:
			DWORD m_idAttach;
			DWORD m_idAttachTo;

			Q_DISABLE_COPY(Cleaner)
			};
			[[maybe_unused]] Cleaner cleaner{windowThreadProcessId, currentThreadId};
			[[maybe_unused]] const auto &cleaner =
			qScopeGuard([windowThreadProcessId, currentThreadId]() {
			::AttachThreadInput(windowThreadProcessId, currentThreadId, FALSE); //
			});

			::BringWindowToTop(hwnd);
			// Activate the window too. This will force us to the virtual desktop this
			@@ -856,6 +845,7 @@
			g_wndProcHash->insert(hWnd, ctx);
			}

			template <bool Destroyed = true>
			static inline void removeManagedWindow(HWND hWnd) {
			// Remove window handle mapping
			if (!g_wndProcHash->remove(hWnd))
			@@ -864,6 +854,11 @@
			// Remove event filter if the all windows has been destroyed
			if (g_wndProcHash->empty()) {
			WindowsNativeEventFilter::uninstall();
			}

			// Restore window proc
			if constexpr (!Destroyed) {
			::SetWindowLongPtrW(hWnd, GWLP_WNDPROC, reinterpret_cast<LONG_PTR>(g_qtWindowProc));
			}
			}

			@@ -922,7 +917,7 @@
			}
			}

			break;
			return;
			}

			case DefaultColorsHook: {
			@@ -975,10 +970,8 @@
			return;
			}

			default: {
			// unreachable
			default:
			break;
			}
			}
			AbstractWindowContext::virtual_hook(id, data);
			}
			@@ -991,8 +984,19 @@
			return getWindowFrameBorderThickness(reinterpret_cast<HWND>(windowId));
			}

			void Win32WindowContext::winIdChanged(QWindow *oldWindow) {
			removeManagedWindow(reinterpret_cast<HWND>(windowId));
			void Win32WindowContext::winIdChanged(QWindow *oldWindow, bool isDestroyed) {
			Q_UNUSED(isDestroyed)

			// If the original window id is valid, remove all resources related
			if (windowId) {
			if (isDestroyed) {
			removeManagedWindow(reinterpret_cast<HWND>(windowId));
			} else {
			removeManagedWindow<false>(reinterpret_cast<HWND>(windowId));
			}
			windowId = 0;
			}

			if (!m_windowHandle) {
			return;
			}
			@@ -1012,7 +1016,7 @@
			#endif

			// Inform Qt we want and have set custom margins
			updateInternalWindowFrameMargins(hWnd, m_windowHandle);
			updateInternalWindowFrameMargins(hWnd, m_windowHandle); // TODO: Restore?

			// Add managed window
			addManagedWindow(hWnd, this);
			@@ -1279,6 +1283,53 @@
			const WindowPart currentWindowPart = lastHitTestResult;
			if (message == WM_NCMOUSEMOVE) {
			if (currentWindowPart != WindowPart::ChromeButton) {
			// https://github.com/qt/qtbase/blob/e26a87f1ecc40bc8c6aa5b889fce67410a57a702/src/widgets/kernel/qwidgetwindow.cpp#L472
			// When the mouse press event arrives, QWidgetWindow will implicitly grab
			// the top widget right under the mouse, and set `qt_button_down` to this
			// widget. At this time, no other widgets will accept the mouse event until
			// QWidgetWindow receives the mouse release event, then set `qt_button_down`
			// to null.

			// Imagine the following situation, now the main window has a pop-up menu,
			// the focus is not on the main window, if we click outside the pop-up menu,
			// the menu will close, which seems to be completely fine. But if we close
			// the menu by clicking on the title bar draggable area, then other widgets
			// won't accept the mouse message afterwards.

			// Here's the reason.
			// When the mouse is placed in the draggable area of the title bar, there
			// are two situations.

			// 1. If the focus is on the main window, and the last result of
			// WM_NCHITTEST is HTCAPTION, the mouse click event in the title bar is
			// taken over by Windows and Qt does not receive the mouse click event.

			// 2. If the main window has a pop-up menu, it is completely different. When
			// the mouse is pressed on the title bar, Windows sends the WM_LBUTTONDOWN
			// message to the window plane of the pop-up menu, the menu is closed, but
			// Qt will continue to forward the event to the QWidget under the mouse, and
			// the event will be processed by QWidgetWindow, causing the title bar
			// widget to be implicitly grabbed. After the menu is closed, Windows
			// immediately sends WM_NCHITTEST, because the mouse is in the title bar
			// draggable area, the result is HTCAPTION, so when the mouse is released,
			// Windows sends WM_NCLBUTTONUP, which is a non-client message, and it
			// will be ignored by Qt. As a consequence, QWidgetWindow can't receive a
			// mouse release message in the client area, so the grab remains, and other
			// widgets cannot receive mouse events.

			// Since we didn't watch the menu window, we cannot capture any mouse
			// press events sent by Windows, so we cannot solve this problem by
			// recording mouse events. Fortunately, we found that the main window will
			// receive a WM_NCMOUSEMOVE message immediately after the menu is closed, so
			// we just manually send a mouse release event when this message arrives and
			// set qt_button_down to null. Don't worry, when receiving WM_NCMOUSEMOVE,
			// there shouldn't be any control in the state of being grabbed.

			// In the native window, although QWidgetWindow handles the forwarded mouse
			// press event when the menu is closed, since the native title bar is not a
			// QWidget, no widget will be grabbed, and `qt_button_down` remains empty,
			// the above problems would not arise.

			m_delegate->resetQtGrabbedControl(m_host);
			if (mouseLeaveBlocked) {
			emulateClientAreaMessage(hWnd, message, wParam, lParam,
			@@ -1341,6 +1392,8 @@
			// window from client area, which means we will get previous window part as
			// HTCLIENT if the mouse leaves window from client area and enters window
			// from non-client area, but it has no bad effect.

			// Why do we need to call this function here?
			m_delegate->resetQtGrabbedControl(m_host);
			}
			}
			@@ -1450,21 +1503,9 @@
			// Terminal does, however, later I found that if we choose a proper
			// color, our homemade top border can almost have exactly the same
			// appearance with the system's one.
			struct HitTestRecorder {
			HitTestRecorder(Win32WindowContext ctx, LRESULT result)
			: ctx(ctx), result(result) {
			}
			~HitTestRecorder() {
			ctx->lastHitTestResult = getHitWindowPart(int(*result));
			}

			private:
			Win32WindowContext *ctx;
			LRESULT *result;

			Q_DISABLE_COPY(HitTestRecorder)
			};
			[[maybe_unused]] HitTestRecorder hitTestRecorder{this, result};
			[[maybe_unused]] const auto &hitTestRecorder = qScopeGuard([this, result]() {
			lastHitTestResult = getHitWindowPart(int(*result)); //
			});

			POINT nativeGlobalPos{GET_X_LPARAM(lParam), GET_Y_LPARAM(lParam)};
			POINT nativeLocalPos = nativeGlobalPos;
			@@ -1584,7 +1625,7 @@
			*result = (isTitleBar ? HTCAPTION : HTCLIENT);
			return true;
			}
			// At this point, we know that the cursor is inside the client area
			// At this point, we know that the cursor is inside the client area,
			// so it has to be either the little border at the top of our custom
			// title bar or the drag bar. Apparently, it must be the drag bar or
			// the little border at the top which the user can use to move or