From e4c90afe8e92f7431ff1568c90b1e1ef20e39087 Mon Sep 17 00:00:00 2001
From: Yuhang Zhao <zhaoyuhang@rankyee.com>
Date: 周一, 18 11月 2024 11:21:56 +0800
Subject: [PATCH] Update README
---
src/core/contexts/win32windowcontext.cpp | 22 +++++++++++++---------
1 files changed, 13 insertions(+), 9 deletions(-)
diff --git a/src/core/contexts/win32windowcontext.cpp b/src/core/contexts/win32windowcontext.cpp
index 9e4155f..8ad7ae8 100644
--- a/src/core/contexts/win32windowcontext.cpp
+++ b/src/core/contexts/win32windowcontext.cpp
@@ -343,7 +343,6 @@
case HTBORDER:
return Win32WindowContext::FixedBorder;
default:
- Q_UNREACHABLE();
break;
}
return Win32WindowContext::Outside;
@@ -835,9 +834,7 @@
}
if (key == QStringLiteral("title-bar-height")) {
- return m_windowId
- ? int(getTitleBarHeight(reinterpret_cast<HWND>(m_windowId)))
- : 0;
+ return m_windowId ? int(getTitleBarHeight(reinterpret_cast<HWND>(m_windowId))) : 0;
}
return AbstractWindowContext::windowAttribute(key);
}
@@ -1696,7 +1693,8 @@
// outside the window, that is, the three transparent window resize area.
// Returning HTCLIENT will confuse Windows, we can't put our controls there
// anyway.
- *result = HTNOWHERE; // Make sure we can know we don't set any value explicitly later.
+ *result = HTNOWHERE; // Make sure we can know we don't set any value
+ // explicitly later.
if (originalHitTestResult == HTCAPTION) {
} else if (isFixedSize || dontOverrideCursor) {
*result = HTBORDER;
@@ -1725,11 +1723,13 @@
} else {
*result = HTLEFT;
}
- } else if (originalHitTestResult == HTLEFT || originalHitTestResult == HTRIGHT) {
+ } else if (originalHitTestResult == HTLEFT ||
+ originalHitTestResult == HTRIGHT) {
if (isFixedWidth) {
*result = HTBORDER;
}
- } else if (originalHitTestResult == HTTOP || originalHitTestResult == HTBOTTOM) {
+ } else if (originalHitTestResult == HTTOP ||
+ originalHitTestResult == HTBOTTOM) {
if (isFixedHeight) {
*result = HTBORDER;
}
@@ -1758,7 +1758,8 @@
// inside our homemade title bar now, return HTCLIENT to let our
// title bar can still capture mouse events.
*result = [&]() {
- if (isFixedSize || isFixedHeight || dontOverrideCursor || (isFixedWidth && (isInLeftBorder || isInRightBorder))) {
+ if (isFixedSize || isFixedHeight || dontOverrideCursor ||
+ (isFixedWidth && (isInLeftBorder || isInRightBorder))) {
if (isInTitleBar) {
return HTCAPTION;
} else {
@@ -2174,7 +2175,10 @@
// of the upper-left non-client area. It's confirmed that this issue exists
// from Windows 7 to Windows 10. Not tested on Windows 11 yet. Don't know
// whether it exists on Windows XP to Windows Vista or not.
- *result = wParam ? WVR_REDRAW : FALSE;
+
+ // https://github.com/chromium/chromium/blob/5d297da3cf2a642e9ace2b23fed097370bc70814/ui/views/win/hwnd_message_handler.cc#L2330
+ // Do not return WVR_REDRAW otherwise child HWNDs will be mispositioned.
+ *result = FALSE;
return true;
}
--
Gitblit v1.9.1