From a51b5688e1c33d3ce96b48c869603b00f908f513 Mon Sep 17 00:00:00 2001
From: Sine Striker <trueful@163.com>
Date: 周三, 13 12月 2023 04:16:37 +0800
Subject: [PATCH] Make better code structure for border handlers

---
 src/widgets/widgetwindowagent.cpp |   81 +---------------------------------------
 1 files changed, 3 insertions(+), 78 deletions(-)

diff --git a/src/widgets/widgetwindowagent.cpp b/src/widgets/widgetwindowagent.cpp
index ea8f6df..ff76aef 100644
--- a/src/widgets/widgetwindowagent.cpp
+++ b/src/widgets/widgetwindowagent.cpp
@@ -9,76 +9,6 @@
 
 namespace QWK {
 
-    class WidgetBorderHandler : public QObject {
-    public:
-        WidgetBorderHandler(QWidget *widget, AbstractWindowContext *ctx)
-            : widget(widget), ctx(ctx), m_thickness(0) {
-            updateThickness();
-            widget->installEventFilter(this);
-        }
-
-        void updateThickness() {
-            // Query thickness
-            bool native = false;
-            void *a[] = {
-                &native,
-                &m_thickness,
-            };
-            ctx->virtual_hook(AbstractWindowContext::BorderThicknessHook, &a);
-        }
-
-        void updateMargins() {
-            if (widget->windowState() & (Qt::WindowMaximized | Qt::WindowFullScreen)) {
-                widget->setContentsMargins({});
-            } else {
-                widget->setContentsMargins({0, int(m_thickness), 0, 0});
-            }
-        }
-
-    protected:
-        bool eventFilter(QObject *obj, QEvent *event) override {
-            switch (event->type()) {
-                case QEvent::Paint: {
-                    if (widget->windowState() & (Qt::WindowMaximized | Qt::WindowFullScreen))
-                        break;
-
-                    auto paintEvent = static_cast<QPaintEvent *>(event);
-                    QPainter painter(widget);
-                    QRect rect = paintEvent->rect();
-                    QRegion region = paintEvent->region();
-                    void *args[] = {
-                        &painter,
-                        &rect,
-                        &region,
-                    };
-                    ctx->virtual_hook(AbstractWindowContext::DrawBordersHook, args);
-                    return true;
-                }
-
-                case QEvent::WindowStateChange: {
-                    updateMargins();
-                    break;
-                }
-
-                case QEvent::WindowActivate:
-                case QEvent::WindowDeactivate: {
-                    widget->update();
-                    break;
-                }
-
-                    // TODO: Handle DPI Change
-
-                default:
-                    break;
-            }
-            return false;
-        }
-
-        QWidget *widget;
-        AbstractWindowContext *ctx;
-        quint32 m_thickness;
-    };
-
     WidgetWindowAgentPrivate::WidgetWindowAgentPrivate() {
     }
 
@@ -114,14 +44,9 @@
         }
         d->hostWidget = w;
 
-        // Install painting hook
-        if (bool needPaintBorder = false;
-            d->context->virtual_hook(AbstractWindowContext::NeedsDrawBordersHook, &needPaintBorder),
-            needPaintBorder) {
-            auto borderHandler = std::make_unique<WidgetBorderHandler>(w, d->context.get());
-            borderHandler->updateMargins();
-            d->borderHandler = std::move(borderHandler);
-        }
+#ifdef Q_OS_WINDOWS
+        d->setupWindows10BorderWorkaround();
+#endif
         return true;
     }
 

--
Gitblit v1.9.1