From 187f82bb4065589fa7fba5f6fa9bd51113357a2c Mon Sep 17 00:00:00 2001
From: SineStriker <trueful@163.com>
Date: 周一, 18 12月 2023 14:29:35 +0800
Subject: [PATCH] minor tweaks

---
 src/core/contexts/abstractwindowcontext.cpp |  118 ++++++++++++++++++++++-------------------------------------
 1 files changed, 44 insertions(+), 74 deletions(-)

diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp
index d3e8c0f..ef1efbf 100644
--- a/src/core/contexts/abstractwindowcontext.cpp
+++ b/src/core/contexts/abstractwindowcontext.cpp
@@ -7,38 +7,20 @@
 
 namespace QWK {
 
-    QWK_USED static constexpr const struct {
-        const quint32 activeLight = MAKE_RGBA_COLOR(210, 233, 189, 226);
-        const quint32 activeDark = MAKE_RGBA_COLOR(177, 205, 190, 240);
-        const quint32 inactiveLight = MAKE_RGBA_COLOR(193, 195, 211, 203);
-        const quint32 inactiveDark = MAKE_RGBA_COLOR(240, 240, 250, 255);
-    } kSampleColorSet;
-
     AbstractWindowContext::AbstractWindowContext() = default;
 
     AbstractWindowContext::~AbstractWindowContext() = default;
 
-    bool AbstractWindowContext::setup(QObject *host, WindowItemDelegate *delegate) {
-        if (!host || !delegate) {
-            return false;
+    void AbstractWindowContext::setup(QObject *host, WindowItemDelegate *delegate) {
+        if (m_host || !host || !delegate) {
+            return;
         }
-
-        auto windowHandle = delegate->hostWindow(host);
-        if (!windowHandle) {
-            return false;
-        }
-
         m_host = host;
         m_delegate.reset(delegate);
-        m_windowHandle = windowHandle;
-
-        if (!setupHost()) {
-            m_host = nullptr;
-            m_delegate.reset();
-            m_windowHandle = nullptr;
-            return false;
+        m_windowHandle = m_delegate->hostWindow(m_host);
+        if (m_windowHandle) {
+            winIdChanged(nullptr);
         }
-        return true;
     }
 
     bool AbstractWindowContext::setHitTestVisible(const QObject *obj, bool visible) {
@@ -52,21 +34,6 @@
         } else {
             m_hitTestVisibleItems.remove(obj);
         }
-        return true;
-    }
-
-    bool AbstractWindowContext::setHitTestVisible(const QRect &rect, bool visible) {
-        Q_ASSERT(rect.isValid());
-        if (!rect.isValid()) {
-            return false;
-        }
-
-        if (visible) {
-            m_hitTestVisibleRects.append(rect);
-        } else {
-            m_hitTestVisibleRects.removeAll(rect);
-        }
-        hitTestVisibleShapeDirty = true;
         return true;
     }
 
@@ -98,20 +65,12 @@
         return true;
     }
 
-    void AbstractWindowContext::showSystemMenu(const QPoint &pos) {
-        virtual_hook(ShowSystemMenuHook, &const_cast<QPoint &>(pos));
+#ifdef Q_OS_MAC
+    void AbstractWindowContext::setSystemButtonArea(const QRect &rect) {
+        m_systemButtonArea = rect;
+        virtual_hook(SystemButtonAreaChangedHook, nullptr);
     }
-
-    QRegion AbstractWindowContext::hitTestShape() const {
-        if (hitTestVisibleShapeDirty) {
-            hitTestVisibleShape = {};
-            for (const auto &rect : m_hitTestVisibleRects) {
-                hitTestVisibleShape += rect;
-            }
-            hitTestVisibleShapeDirty = false;
-        }
-        return hitTestVisibleShape;
-    }
+#endif
 
     bool AbstractWindowContext::isInSystemButtons(const QPoint &pos,
                                                   WindowAgentBase::SystemButton *button) const {
@@ -168,9 +127,6 @@
             }
         }
 
-        if (!m_hitTestVisibleRects.isEmpty() && hitTestShape().contains(pos)) {
-            return false;
-        }
         return true;
     }
 
@@ -178,30 +134,32 @@
         return {};
     }
 
+    QWK_USED static constexpr const struct {
+        const quint32 activeLight = MAKE_RGBA_COLOR(210, 233, 189, 226);
+        const quint32 activeDark = MAKE_RGBA_COLOR(177, 205, 190, 240);
+        const quint32 inactiveLight = MAKE_RGBA_COLOR(193, 195, 211, 203);
+        const quint32 inactiveDark = MAKE_RGBA_COLOR(240, 240, 250, 255);
+    } kSampleColorSet;
+
     void AbstractWindowContext::virtual_hook(int id, void *data) {
         switch (id) {
-            case NeedsDrawBordersHook: {
-                auto &result = *static_cast<bool *>(data);
-                result = false;
-                return;
+            case CentralizeHook: {
+                // TODO: Qt
+                break;
             }
 
-            case BorderThicknessHook: {
-                auto args = static_cast<void **>(data);
-                const bool requireNative = *static_cast<const bool *>(args[0]);
-                quint32 &thickness = *static_cast<quint32 *>(args[1]);
-                std::ignore = requireNative;
-                thickness = 1;
-                return;
+            case ShowSystemMenuHook: {
+                // TODO: Qt
+                break;
             }
 
-            case BorderColorsHook: {
-                auto arr = *reinterpret_cast<QList<QColor> *>(data);
-                arr.clear();
-                arr.push_back(kSampleColorSet.activeLight);
-                arr.push_back(kSampleColorSet.activeDark);
-                arr.push_back(kSampleColorSet.inactiveLight);
-                arr.push_back(kSampleColorSet.inactiveDark);
+            case DefaultColorsHook: {
+                auto &map = *static_cast<QMap<QString, QColor> *>(data);
+                map.clear();
+                map.insert(QStringLiteral("activeLight"), kSampleColorSet.activeLight);
+                map.insert(QStringLiteral("activeDark"), kSampleColorSet.activeDark);
+                map.insert(QStringLiteral("inactiveLight"), kSampleColorSet.inactiveLight);
+                map.insert(QStringLiteral("inactiveDark"), kSampleColorSet.inactiveDark);
                 return;
             }
 
@@ -210,4 +168,16 @@
         }
     }
 
-}
\ No newline at end of file
+    void AbstractWindowContext::showSystemMenu(const QPoint &pos) {
+        virtual_hook(ShowSystemMenuHook, &const_cast<QPoint &>(pos));
+    }
+
+    void AbstractWindowContext::notifyWinIdChange() {
+        auto oldWindow = m_windowHandle;
+        m_windowHandle = m_delegate->window(m_host);
+        if (oldWindow == m_windowHandle)
+            return;
+        winIdChanged(oldWindow);
+    }
+
+}

--
Gitblit v1.9.1