From cc65c4da6ad762e8648b2ab6a801e07c50922d3d Mon Sep 17 00:00:00 2001
From: Zhao Yuhang <2546789017@qq.com>
Date: 周二, 04 2月 2025 21:58:21 +0800
Subject: [PATCH] fix compilation error
---
src/widgets/widgetwindowagent_win.cpp | 156 +++++++++++++++++++++------------------------------
1 files changed, 64 insertions(+), 92 deletions(-)
diff --git a/src/widgets/widgetwindowagent_win.cpp b/src/widgets/widgetwindowagent_win.cpp
index f192f6b..2e08d04 100644
--- a/src/widgets/widgetwindowagent_win.cpp
+++ b/src/widgets/widgetwindowagent_win.cpp
@@ -8,8 +8,11 @@
#include <QtCore/QDateTime>
#include <QtGui/QPainter>
+#include <QtCore/private/qcoreapplication_p.h>
+
#include <QWKCore/qwindowkit_windows.h>
#include <QWKCore/private/qwkglobal_p.h>
+#include <QWKCore/private/windows10borderhandler_p.h>
namespace QWK {
@@ -37,122 +40,92 @@
// returns, because Qt calls BeginPaint() and EndPaint() itself. We should make sure that we
// draw the top border between these two calls, otherwise some display exceptions may arise.
- class WidgetBorderHandler : public QObject, public NativeEventFilter, public SharedEventFilter {
+ class WidgetBorderHandler : public QObject, public Windows10BorderHandler {
public:
explicit WidgetBorderHandler(QWidget *widget, AbstractWindowContext *ctx,
QObject *parent = nullptr)
- : QObject(parent), widget(widget), ctx(ctx) {
+ : QObject(parent), Windows10BorderHandler(ctx), widget(widget) {
widget->installEventFilter(this);
- // https://github.com/microsoft/terminal/blob/71a6f26e6ece656084e87de1a528c4a8072eeabd/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp#L940
- // Must extend top frame to client area
- static QVariant defaultMargins = QVariant::fromValue(QMargins(0, 1, 0, 0));
- ctx->setWindowAttribute(QStringLiteral("extra-margins"), defaultMargins);
-
- // Enable dark mode by default, otherwise the system borders are white
- ctx->setWindowAttribute(QStringLiteral("dark-mode"), true);
-
- ctx->installNativeEventFilter(this);
- ctx->installSharedEventFilter(this);
-
- updateGeometry();
- }
-
- inline bool isNormalWindow() const {
- return !(widget->windowState() &
- (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen));
- }
-
- inline void updateGeometry() {
- if (isNormalWindow()) {
- widget->setContentsMargins(
- {0, ctx->windowAttribute(QStringLiteral("border-thickness")).toInt(), 0, 0});
- } else {
- widget->setContentsMargins({});
+ // First update
+ if (ctx->windowId()) {
+ setupNecessaryAttributes();
}
+ WidgetBorderHandler::updateGeometry();
}
- inline void resumeWidgetEventAndDraw(QWidget *w, QEvent *event) {
- // Let the widget paint first
- Private::ObjectHelper::sendEvent(w, event);
+ void updateGeometry() override {
+ widget->setContentsMargins(isNormalWindow() ? QMargins(0, borderThickness(), 0, 0)
+ : QMargins());
+ }
+
+ bool isWindowActive() const override {
+ return widget->isActiveWindow();
+ }
+
+ inline void forwardEventToWidgetAndDraw(QWidget *w, QEvent *event) {
+ // https://github.com/qt/qtbase/blob/e26a87f1ecc40bc8c6aa5b889fce67410a57a702/src/widgets/kernel/qapplication.cpp#L3286
+ // Deliver the event
+ if (!forwardObjectEventFilters(this, w, event)) {
+ // Let the widget paint first
+ std::ignore = static_cast<QObject *>(w)->event(event);
+ QCoreApplicationPrivate::setEventSpontaneous(event, false);
+ }
// Due to the timer or user action, Qt will repaint some regions spontaneously,
// even if there is no WM_PAINT message, we must wait for it to finish painting
// and then update the top border area.
- ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook2, nullptr);
+ drawBorder();
}
- inline void resumeWindowEventAndDraw(QWindow *window, QEvent *event) {
- // Let Qt paint first
- Private::ObjectHelper::sendEvent(window, event);
+ inline void forwardEventToWindowAndDraw(QWindow *window, QEvent *event) {
+ // https://github.com/qt/qtbase/blob/e26a87f1ecc40bc8c6aa5b889fce67410a57a702/src/widgets/kernel/qapplication.cpp#L3286
+ // Deliver the event
+ if (!forwardObjectEventFilters(ctx, window, event)) {
+ // Let Qt paint first
+ std::ignore = static_cast<QObject *>(window)->event(event);
+ QCoreApplicationPrivate::setEventSpontaneous(event, false);
+ }
// Upon receiving the WM_PAINT message, Qt will repaint the entire view, and we
// must wait for it to finish painting before drawing this top border area.
- ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook2, nullptr);
- }
-
- inline void updateExtraMargins(bool windowActive) {
- if (windowActive) {
- // Restore margins when the window is active
- static QVariant defaultMargins = QVariant::fromValue(QMargins(0, 1, 0, 0));
- ctx->setWindowAttribute(QStringLiteral("extra-margins"), defaultMargins);
- return;
- }
-
- // https://github.com/microsoft/terminal/blob/71a6f26e6ece656084e87de1a528c4a8072eeabd/src/cascadia/WindowsTerminal/NonClientIslandWindow.cpp#L904
- // When the window is inactive, there is a transparency bug in the top
- // border, and we need to extend the non-client area to the whole title
- // bar.
- QRect frame = ctx->windowAttribute(QStringLiteral("window-rect")).toRect();
- QMargins margins{0, -frame.top(), 0, 0};
- ctx->setWindowAttribute(QStringLiteral("extra-margins"), QVariant::fromValue(margins));
+ drawBorder();
}
protected:
- bool nativeEventFilter(const QByteArray &eventType, void *message,
- QT_NATIVE_EVENT_RESULT_TYPE *result) override {
- Q_UNUSED(eventType)
-
- const auto msg = static_cast<const MSG *>(message);
- switch (msg->message) {
- case WM_DPICHANGED: {
- updateGeometry();
- updateExtraMargins(widget->isActiveWindow());
- break;
- }
-
- case WM_ACTIVATE: {
- updateExtraMargins(LOWORD(msg->wParam) != WA_INACTIVE);
- break;
- }
-
- default:
- break;
- }
- return false;
- }
-
bool sharedEventFilter(QObject *obj, QEvent *event) override {
Q_UNUSED(obj)
- auto window = widget->windowHandle();
+ switch (event->type()) {
+ case QEvent::Expose: {
+ // Qt will absolutely send a QExposeEvent or QResizeEvent to the QWindow when it
+ // receives a WM_PAINT message. When the control flow enters the expose handler,
+ // Qt must have already called BeginPaint() and it's the best time for us to
+ // draw the top border.
- // Qt will absolutely send a QExposeEvent or QResizeEvent to the QWindow when it
- // receives a WM_PAINT message. When the control flow enters the expose handler, Qt
- // must have already called BeginPaint() and it's the best time for us to draw the
- // top border.
-
- // Since a QExposeEvent will be sent immediately after the QResizeEvent, we can simply
- // ignore it.
- if (event->type() == QEvent::Expose) {
- auto ee = static_cast<QExposeEvent *>(event);
- if (window->isExposed() && isNormalWindow() && !ee->region().isNull()) {
- resumeWindowEventAndDraw(window, event);
- return true;
+ // Since a QExposeEvent will be sent immediately after the QResizeEvent, we can
+ // simply ignore it.
+#if (QT_VERSION >= QT_VERSION_CHECK(6, 0, 0))
+ struct ExposeEvent : public QExposeEvent {
+ inline const QRegion &getRegion() const { return m_region; }
+ };
+ auto ee = static_cast<ExposeEvent *>(event);
+ bool exposeRegionValid = !ee->getRegion().isNull();
+#else
+ auto ee = static_cast<QExposeEvent *>(event);
+ bool exposeRegionValid = !ee->region().isNull();
+#endif
+ auto window = widget->windowHandle();
+ if (window->isExposed() && isNormalWindow() && exposeRegionValid) {
+ forwardEventToWindowAndDraw(window, event);
+ return true;
+ }
+ break;
}
+ default:
+ break;
}
-
- return false;
+ return Windows10BorderHandler::sharedEventFilter(obj, event);
}
bool eventFilter(QObject *obj, QEvent *event) override {
@@ -162,7 +135,7 @@
case QEvent::UpdateRequest: {
if (!isNormalWindow())
break;
- resumeWidgetEventAndDraw(widget, event);
+ forwardEventToWidgetAndDraw(widget, event);
return true;
}
@@ -184,7 +157,6 @@
}
QWidget *widget;
- AbstractWindowContext *ctx;
};
void WidgetWindowAgentPrivate::setupWindows10BorderWorkaround() {
--
Gitblit v1.9.1