From 58afa6aa95b54c5a6db524669f5c753060b11c67 Mon Sep 17 00:00:00 2001
From: Yuhang Zhao <zhaoyuhang@rankyee.com>
Date: Sat, 19 Aug 2023 15:45:49 +0800
Subject: [PATCH] snap layout: try improve the maximize button handling

---
 src/core/utils.cpp | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/core/utils.cpp b/src/core/utils.cpp
index 5dab0f9c..31581c42 100644
--- a/src/core/utils.cpp
+++ b/src/core/utils.cpp
@@ -731,18 +731,18 @@ void Utils::emulateQtMouseEvent(const QObject *target, const QWindow *window, co
         if (!obj) {
             return;
         }
-        static constexpr const auto fakePos = QPoint{ -999, -999 };
-        const QPoint tweakedLocalPos = (fake ? fakePos : localPos);
-        const QPoint tweakedScenePos = (fake ? fakePos : scenePos);
-        const QPoint tweakedGlobalPos = (fake ? fakePos : globalPos);
-        QMouseEvent event(QEvent::MouseButtonRelease, tweakedLocalPos, tweakedScenePos, tweakedGlobalPos, button, buttons, modifiers);
+        QMouseEvent event(QEvent::MouseButtonRelease, localPos, scenePos, globalPos, button, buttons, modifiers);
         QCoreApplication::sendEvent(obj, &event);
     };
     switch (buttonState) {
     case ButtonState::Normal: {
         targetObj->setProperty(kEnteredFlag, {});
         // Send an extra mouse release event to let the control dismiss it's hover state.
-        sendMouseReleaseEvent(targetObj, true);
+        // But only when the mouse is not still hovering above the control, otherwise Qt will
+        // generate a mouse click event, which is not what the user would expect.
+        if (!underMouse) {
+            sendMouseReleaseEvent(targetObj);
+        }
         if (isWidget) {
             sendLeaveEvent(targetObj);
         } else {