From 9dcea027f16c4ce5852da3dfc8aca284c582bd7d Mon Sep 17 00:00:00 2001
From: sola.lu <sola.lu.greentest.com.cn>
Date: 周五, 06 6月 2025 18:19:18 +0800
Subject: [PATCH] 1.注释掉custom margin,解决程序放大后,遮盖窗口底部内容的问题。

---
 src/quick/quickwindowagent_win.cpp |  131 +++++++++++++++++++++++++++----------------
 1 files changed, 82 insertions(+), 49 deletions(-)

diff --git a/src/quick/quickwindowagent_win.cpp b/src/quick/quickwindowagent_win.cpp
index 338dbfa..712e742 100644
--- a/src/quick/quickwindowagent_win.cpp
+++ b/src/quick/quickwindowagent_win.cpp
@@ -8,44 +8,69 @@
 #include <QtQuick/private/qquickitem_p.h>
 
 #include <QWKCore/qwindowkit_windows.h>
+#include <QWKCore/private/windows10borderhandler_p.h>
 
 namespace QWK {
 
-#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDERS)
-    // TODO: Find a way to draw native border
-    // We haven't found a way to place hooks in the Quick program and call the GDI API to draw
-    // the native border area so that we'll use the emulated drawn border for now.
+    static inline bool isWindows1022H2OrGreater() {
+        RTL_OSVERSIONINFOW rovi = Private::GetRealOSVersion();
+        return (rovi.dwMajorVersion > 10) ||
+               (rovi.dwMajorVersion == 10 &&
+                (rovi.dwMinorVersion > 0 || rovi.dwBuildNumber >= 19045));
+    }
 
-    class BorderItem : public QQuickPaintedItem,
-                       public NativeEventFilter,
-                       public SharedEventFilter {
+#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDERS)
+
+    class BorderItem : public QQuickPaintedItem, public Windows10BorderHandler {
     public:
         explicit BorderItem(QQuickItem *parent, AbstractWindowContext *context);
         ~BorderItem() override;
 
-        inline bool isNormalWindow() const;
-
-        inline void updateGeometry();
+        bool shouldEnableEmulatedPainter() const;
+        void updateGeometry() override;
 
     public:
         void paint(QPainter *painter) override;
         void itemChange(ItemChange change, const ItemChangeData &data) override;
 
     protected:
+        bool sharedEventFilter(QObject *obj, QEvent *event) override;
         bool nativeEventFilter(const QByteArray &eventType, void *message,
                                QT_NATIVE_EVENT_RESULT_TYPE *result) override;
 
-        bool sharedEventFilter(QObject *obj, QEvent *event) override;
-
-        AbstractWindowContext *context;
-
     private:
+        volatile bool needPaint = false;
+
+        void _q_afterSynchronizing();
         void _q_windowActivityChanged();
     };
 
+    bool BorderItem::shouldEnableEmulatedPainter() const {
+#  if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
+        const QQuickWindow* win = window();
+        if (!win) {
+            return true;
+        }
+        auto api = win->rendererInterface()->graphicsApi();
+        switch (api) {
+            case QSGRendererInterface::OpenGL:
+                // FIXME: experimental, try to find the exact fixed version.
+                return !isWindows1022H2OrGreater();
+            case QSGRendererInterface::Direct3D11:
+#    if QT_VERSION >= QT_VERSION_CHECK(6, 6, 0)
+            case QSGRendererInterface::Direct3D12:
+#    endif
+                return false;
+            default:
+                break;
+        }
+#  endif
+        return true;
+    }
+
     BorderItem::BorderItem(QQuickItem *parent, AbstractWindowContext *context)
-        : QQuickPaintedItem(parent), context(context) {
-        setAntialiasing(true);   // We needs anti-aliasing to give us better result.
+        : QQuickPaintedItem(parent), Windows10BorderHandler(context) {
+        setAntialiasing(true);   // We need anti-aliasing to give us better result.
         setFillColor({});        // Will improve the performance a little bit.
         setOpaquePainting(true); // Will also improve the performance, we don't draw
                                  // semi-transparent borders of course.
@@ -56,37 +81,41 @@
         anchors->setLeft(parentPri->left());
         anchors->setRight(parentPri->right());
 
-        setZ(9999); // Make sure our fake border always above everything in the window.
+        setZ(std::numeric_limits<qreal>::max()); // Make sure our fake border always above
+                                                 // everything in the window.
 
-        context->installNativeEventFilter(this);
-        context->installSharedEventFilter(this);
-
+#  if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
+        connect(window(), &QQuickWindow::afterSynchronizing, this,
+                &BorderItem::_q_afterSynchronizing, Qt::DirectConnection);
+#  endif
         connect(window(), &QQuickWindow::activeChanged, this,
                 &BorderItem::_q_windowActivityChanged);
-        updateGeometry();
+
+        // First update
+        if (context->windowId()) {
+            setupNecessaryAttributes();
+        }
+        BorderItem::updateGeometry();
     }
 
     BorderItem::~BorderItem() = default;
 
-    bool BorderItem::isNormalWindow() const {
-        return !(context->window()->windowState() &
-                 (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen));
-    }
-
     void BorderItem::updateGeometry() {
-        setHeight(context->windowAttribute(QStringLiteral("border-thickness")).toInt());
+        const QQuickWindow* win = window();
+        if (!win) {
+            return;
+        }
+        setHeight(borderThickness() / win->effectiveDevicePixelRatio());
         setVisible(isNormalWindow());
     }
 
     void BorderItem::paint(QPainter *painter) {
-        QRect rect(QPoint(0, 0), size().toSize());
-        QRegion region(rect);
-        void *args[] = {
-            painter,
-            &rect,
-            &region,
-        };
-        context->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook, args);
+        Q_UNUSED(painter)
+        if (shouldEnableEmulatedPainter()) {
+            drawBorderEmulated(painter, QRect({0, 0}, size().toSize()));
+        } else {
+            needPaint = true;
+        }
     }
 
     void BorderItem::itemChange(ItemChange change, const ItemChangeData &data) {
@@ -102,10 +131,21 @@
         }
     }
 
+    bool BorderItem::sharedEventFilter(QObject *obj, QEvent *event) {
+        Q_UNUSED(obj)
+
+        switch (event->type()) {
+            case QEvent::WindowStateChange: {
+                updateGeometry();
+            }
+            default:
+                break;
+        }
+        return Windows10BorderHandler::sharedEventFilter(obj, event);
+    }
+
     bool BorderItem::nativeEventFilter(const QByteArray &eventType, void *message,
                                        QT_NATIVE_EVENT_RESULT_TYPE *result) {
-        Q_UNUSED(eventType)
-
         const auto msg = static_cast<const MSG *>(message);
         switch (msg->message) {
             case WM_THEMECHANGED:
@@ -125,21 +165,14 @@
             default:
                 break;
         }
-        return false;
+        return Windows10BorderHandler::nativeEventFilter(eventType, message, result);
     }
 
-    bool BorderItem::sharedEventFilter(QObject *obj, QEvent *event) {
-        Q_UNUSED(obj)
-        
-        switch (event->type()) {
-            case QEvent::WindowStateChange: {
-                updateGeometry();
-                break;
-            }
-            default:
-                break;
+    void BorderItem::_q_afterSynchronizing() {
+        if (needPaint) {
+            needPaint = false;
+            drawBorderNative();
         }
-        return false;
     }
 
     void BorderItem::_q_windowActivityChanged() {

--
Gitblit v1.9.1