From 6b2d31247dc2c2804e571b31a71c8a423c1db9d4 Mon Sep 17 00:00:00 2001 From: Sine Striker <trueful@163.com> Date: 周二, 26 12月 2023 01:44:03 +0800 Subject: [PATCH] Totally fix top border issue on Win10 for QtWidgets --- src/quick/quickwindowagent_win.cpp | 41 ++++++++++++++++++++++++++++++++--------- 1 files changed, 32 insertions(+), 9 deletions(-) diff --git a/src/quick/quickwindowagent_win.cpp b/src/quick/quickwindowagent_win.cpp index 8b2ea86..a1966fd 100644 --- a/src/quick/quickwindowagent_win.cpp +++ b/src/quick/quickwindowagent_win.cpp @@ -4,18 +4,20 @@ #include <QtQuick/private/qquickitem_p.h> #include <QWKCore/qwindowkit_windows.h> -#include <QWKCore/qwkconfig.h> -#include <QWKCore/private/nativeeventfilter_p.h> namespace QWK { #if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDER) - class BorderItem : public QQuickPaintedItem, public NativeEventFilter { + class BorderItem : public QQuickPaintedItem, + public NativeEventFilter, + public SharedEventFilter { public: explicit BorderItem(QQuickItem *parent, AbstractWindowContext *context); ~BorderItem() override; - void updateGeometry(); + inline bool isNormalWindow() const; + + inline void updateGeometry(); public: void paint(QPainter *painter) override; @@ -24,6 +26,8 @@ protected: bool nativeEventFilter(const QByteArray &eventType, void *message, QT_NATIVE_EVENT_RESULT_TYPE *result) override; + + bool sharedEventFilter(QObject *obj, QEvent *event) override; AbstractWindowContext *context; @@ -47,6 +51,8 @@ setZ(9999); // Make sure our fake border always above everything in the window. context->installNativeEventFilter(this); + context->installSharedEventFilter(this); + connect(window(), &QQuickWindow::activeChanged, this, &BorderItem::_q_windowActivityChanged); updateGeometry(); @@ -54,8 +60,14 @@ BorderItem::~BorderItem() = default; + bool BorderItem::isNormalWindow() const { + return !(context->window()->windowState() & + (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen)); + } + void BorderItem::updateGeometry() { - setHeight(context->property("borderThickness").toInt()); + setHeight(context->windowAttribute(QStringLiteral("border-thickness")).toInt()); + setVisible(isNormalWindow()); } void BorderItem::paint(QPainter *painter) { @@ -109,19 +121,30 @@ return false; } + bool BorderItem::sharedEventFilter(QObject *obj, QEvent *event) { + Q_UNUSED(obj) + switch (event->type()) { + case QEvent::WindowStateChange: { + updateGeometry(); + break; + } + default: + break; + } + return false; + } + void BorderItem::_q_windowActivityChanged() { update(); } -#endif void QuickWindowAgentPrivate::setupWindows10BorderWorkaround() { -#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDER) // Install painting hook auto ctx = context.get(); - if (ctx->property("needBorderPainter").toBool()) { + if (ctx->windowAttribute(QStringLiteral("win10-border-needed")).toBool()) { std::ignore = new BorderItem(hostWindow->contentItem(), ctx); } -#endif } +#endif } -- Gitblit v1.9.1