From 29901fc2a97eedd3c914f807d1819c9ea7e69973 Mon Sep 17 00:00:00 2001
From: Sine Striker <trueful@163.com>
Date: 周一, 25 12月 2023 17:53:34 +0800
Subject: [PATCH] Optimize Windows 10 border handling
---
src/widgets/widgetwindowagent_win.cpp | 127 ++++++++++++++++++++++++++++++++----------
1 files changed, 97 insertions(+), 30 deletions(-)
diff --git a/src/widgets/widgetwindowagent_win.cpp b/src/widgets/widgetwindowagent_win.cpp
index 58d52dd..9e49d38 100644
--- a/src/widgets/widgetwindowagent_win.cpp
+++ b/src/widgets/widgetwindowagent_win.cpp
@@ -1,48 +1,84 @@
#include "widgetwindowagent_p.h"
+#include <QtCore/QDebug>
+#include <QtCore/QDateTime>
#include <QtGui/QPainter>
-#include <QWKCore/private/eventobserver_p.h>
+#include <QWKCore/qwindowkit_windows.h>
+#include <QWKCore/private/nativeeventfilter_p.h>
namespace QWK {
- class WidgetBorderHandler : public QObject, public EventObserver {
- Q_OBJECT
+#if QWINDOWKIT_CONFIG(ENABLE_WINDOWS_SYSTEM_BORDER)
+
+ class WidgetBorderHandler;
+
+ class WidgetBorderHandler : public QObject, public NativeEventFilter, public SharedEventFilter {
public:
- explicit WidgetBorderHandler(QWidget *widget, AbstractWindowContext *ctx,
- QObject *parent = nullptr)
- : QObject(parent), widget(widget), ctx(ctx) {
+ explicit WidgetBorderHandler(QWidget *widget, AbstractWindowContext *ctx)
+ : QObject(ctx), widget(widget), ctx(ctx) {
widget->installEventFilter(this);
- ctx->addObserver(this);
+ ctx->installNativeEventFilter(this);
+ ctx->installSharedEventFilter(this);
+
updateGeometry();
}
+ inline bool isNormalWindow() const {
+ return !(widget->windowState() &
+ (Qt::WindowMinimized | Qt::WindowMaximized | Qt::WindowFullScreen));
+ }
+
void updateGeometry() {
- if (widget->windowState() & (Qt::WindowMaximized | Qt::WindowFullScreen)) {
- widget->setContentsMargins({});
- } else {
+ if (isNormalWindow()) {
widget->setContentsMargins({
0,
ctx->property("borderThickness").toInt(),
0,
0,
});
+ } else {
+ widget->setContentsMargins({});
}
}
protected:
- bool observe(QEvent *event) override {
- switch (event->type()) {
- case QEvent::UpdateRequest: {
+ 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();
+ break;
+ }
+
+ case WM_THEMECHANGED:
+ case WM_SYSCOLORCHANGE:
+ case WM_DWMCOLORIZATIONCOLORCHANGED: {
widget->update();
break;
}
- case QEvent::ScreenChangeInternal: {
- updateGeometry();
+ case WM_SETTINGCHANGE: {
+ if (!msg->wParam && msg->lParam &&
+ std::wcscmp(reinterpret_cast<LPCWSTR>(msg->lParam), L"ImmersiveColorSet") ==
+ 0) {
+ widget->update();
+ }
break;
}
+# if 0
+ case WM_ACTIVATE: {
+ if (LOWORD(msg->wParam) == WA_INACTIVE) {
+ // 绐楀彛澶卞幓婵�娲荤姸鎬�
+ } else {
+ // 绐楀彛琚縺娲�
+ }
+ break;
+ }
+# endif
default:
break;
@@ -50,23 +86,55 @@
return false;
}
+ bool sharedEventFilter(QObject *obj, QEvent *event) override {
+ Q_UNUSED(obj)
+
+ auto window = widget->windowHandle();
+ if (event->type() == QEvent::Expose) {
+ // Qt will absolutely send a QExposeEvent 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.
+ auto ee = static_cast<QExposeEvent *>(event);
+ if (isNormalWindow() && window->isExposed() && !ee->region().isNull()) {
+ // Friend class helping to call `event`
+ class HackedWindow : public QWindow {
+ public:
+ friend class QWK::WidgetBorderHandler;
+ };
+
+ // Let Qt paint first
+ static_cast<HackedWindow *>(window)->event(event);
+
+ // Upon receiving the WM_PAINT message, Qt will redraw the entire view, and we
+ // must wait for it to finish redrawing before drawing this top border area
+ ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook2, nullptr);
+ return true;
+ }
+ }
+ return false;
+ }
+
bool eventFilter(QObject *obj, QEvent *event) override {
+ Q_UNUSED(obj)
switch (event->type()) {
- case QEvent::Paint: {
- if (widget->windowState() & (Qt::WindowMaximized | Qt::WindowFullScreen))
+ case QEvent::UpdateRequest: {
+ if (!isNormalWindow())
break;
- auto paintEvent = static_cast<QPaintEvent *>(event);
- auto rect = paintEvent->rect();
- auto region = paintEvent->region();
-
- QPainter painter(widget);
- void *args[] = {
- &painter,
- &rect,
- ®ion,
+ // Friend class helping to call `event`
+ class HackedWidget : public QWidget {
+ public:
+ friend class QWK::WidgetBorderHandler;
};
- ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook, args);
+
+ // Let the widget paint first
+ static_cast<HackedWidget *>(widget)->event(event);
+
+ // Due to the timer or user action, Qt will redraw some regions spontaneously,
+ // even if there is no WM_PAINT message, we must wait for it to finish redrawing
+ // and then update the upper border area
+ ctx->virtual_hook(AbstractWindowContext::DrawWindows10BorderHook2, nullptr);
return true;
}
@@ -95,10 +163,9 @@
// Install painting hook
auto ctx = context.get();
if (ctx->property("needBorderPainter").toBool()) {
- std::ignore = new WidgetBorderHandler(hostWidget, ctx, ctx);
+ std::ignore = new WidgetBorderHandler(hostWidget, ctx);
}
}
+#endif
}
-
-#include "widgetwindowagent_win.moc"
\ No newline at end of file
--
Gitblit v1.9.1