From 327055934009dc416be6234db8e4d26fab81fb80 Mon Sep 17 00:00:00 2001
From: SineStriker <55847490+SineStriker@users.noreply.github.com>
Date: 周四, 21 12月 2023 23:21:28 +0800
Subject: [PATCH] Merge pull request #8 from stdware/stylesupport

---
 src/core/contexts/abstractwindowcontext.cpp |   86 ++++++++++++++++++++----------------------
 1 files changed, 41 insertions(+), 45 deletions(-)

diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp
index 535ffc2..0d72e3c 100644
--- a/src/core/contexts/abstractwindowcontext.cpp
+++ b/src/core/contexts/abstractwindowcontext.cpp
@@ -3,10 +3,32 @@
 #include <QtGui/QPen>
 #include <QtGui/QPainter>
 #include <QtGui/QScreen>
+#include <memory>
 
 #include "qwkglobal_p.h"
 
 namespace QWK {
+
+    class WinIdChangeEventFilter : public QObject {
+    public:
+        explicit WinIdChangeEventFilter(QObject *widget, AbstractWindowContext *ctx,
+                                        QObject *parent = nullptr)
+            : QObject(parent), ctx(ctx) {
+            widget->installEventFilter(this);
+        }
+
+    protected:
+        bool eventFilter(QObject *obj, QEvent *event) override {
+            Q_UNUSED(obj)
+            if (event->type() == QEvent::WinIdChange) {
+                ctx->notifyWinIdChange();
+            }
+            return false;
+        }
+
+    protected:
+        AbstractWindowContext *ctx;
+    };
 
     AbstractWindowContext::AbstractWindowContext() = default;
 
@@ -18,22 +40,12 @@
         }
         m_host = host;
         m_delegate.reset(delegate);
-        setEnabled(true);
-    }
+        m_winIdChangeEventFilter = std::make_unique<WinIdChangeEventFilter>(host, this);
 
-    void AbstractWindowContext::setWindowAttribute(const QString &key, const QVariant &var) {
-        auto it = m_windowAttributes.find(key);
-        if (it.value() == var)
-            return;
-
-        auto newVar = var;
-        auto oldVar = it.value();
-        void *args[] = {
-            &const_cast<QString &>(key),
-            &newVar,
-            &oldVar,
-        };
-        virtual_hook(WindowAttributeChangedHook, args);
+        m_windowHandle = m_delegate->hostWindow(m_host);
+        if (m_windowHandle) {
+            winIdChanged();
+        }
     }
 
     bool AbstractWindowContext::setHitTestVisible(const QObject *obj, bool visible) {
@@ -69,6 +81,15 @@
         if (m_titleBar == item) {
             return false;
         }
+
+        if (m_titleBar) {
+            // Since the title bar is changed, all items inside it should be dereferenced right away
+            for (auto &button : m_systemButtons) {
+                button = nullptr;
+            }
+            m_hitTestVisibleItems.clear();
+        }
+
         m_titleBar = item;
         return true;
     }
@@ -152,6 +173,9 @@
     void AbstractWindowContext::virtual_hook(int id, void *data) {
         switch (id) {
             case CentralizeHook: {
+                if (!m_windowHandle)
+                    return;
+
                 QRect screenGeometry = m_windowHandle->screen()->geometry();
                 int x = (screenGeometry.width() - m_windowHandle->width()) / 2;
                 int y = (screenGeometry.height() - m_windowHandle->height()) / 2;
@@ -190,39 +214,11 @@
     }
 
     void AbstractWindowContext::notifyWinIdChange() {
-        if (!m_internalEnabled)
-            return;
-
         auto oldWindow = m_windowHandle;
+        m_windowHandle = m_delegate->hostWindow(m_host);
         if (oldWindow == m_windowHandle)
             return;
-        auto isDestroyed = oldWindow && m_windowHandleCache.isNull();
-        m_windowHandle = m_delegate->window(m_host);
-        m_windowHandleCache = m_windowHandle;
-        winIdChanged(oldWindow, isDestroyed);
-    }
-
-    void AbstractWindowContext::setEnabled(bool enabled) {
-        if (enabled == m_internalEnabled)
-            return;
-        m_internalEnabled = enabled;
-
-        if (enabled) {
-            m_windowHandle = m_delegate->window(m_host);
-            m_windowHandleCache = m_windowHandle;
-            if (m_windowHandle) {
-                winIdChanged(nullptr, false);
-            }
-            return;
-        }
-
-        if (!m_windowHandle)
-            return;
-
-        auto oldWindow = m_windowHandle;
-        m_windowHandle = nullptr;
-        m_windowHandleCache.clear();
-        winIdChanged(oldWindow, false);
+        winIdChanged();
     }
 
 }

--
Gitblit v1.9.1