From a5d13e19dd7f6037e10b649c49805922ae5e0fa6 Mon Sep 17 00:00:00 2001 From: SineStriker <trueful@163.com> Date: 周五, 22 12月 2023 17:04:29 +0800 Subject: [PATCH] Prepare to remove style support again --- src/core/contexts/cocoawindowcontext.mm | 105 ++++++++++++++++++++++++++++++---------------------- 1 files changed, 61 insertions(+), 44 deletions(-) diff --git a/src/core/contexts/cocoawindowcontext.mm b/src/core/contexts/cocoawindowcontext.mm index aec1f20..949681d 100644 --- a/src/core/contexts/cocoawindowcontext.mm +++ b/src/core/contexts/cocoawindowcontext.mm @@ -5,15 +5,10 @@ #include <QtGui/QGuiApplication> -namespace QWK { +#include "qwkglobal_p.h" +#include "systemwindow_p.h" -#if QT_VERSION < QT_VERSION_CHECK(6, 0, 0) - static const struct QWK_Hook { - QWK_Hook() { - qputenv("QT_MAC_WANTS_LAYER", "1"); - } - } g_hook{}; -#endif +namespace QWK { struct NSWindowProxy { NSWindowProxy(NSWindow *macWindow) { @@ -104,8 +99,8 @@ nswindow.hasShadow = YES; nswindow.showsToolbarButton = NO; nswindow.movableByWindowBackground = NO; - // nswindow.movable = NO; // This line causes the window in the wrong position when - // become fullscreen. + nswindow.movable = NO; // This line causes the window in the wrong position when + // become fullscreen. // For some unknown reason, we don't need the following hack in Qt versions below or // equal to 6.2.4. #if (QT_VERSION > QT_VERSION_CHECK(6, 2, 4)) @@ -215,15 +210,6 @@ return [nsview window]; } - static inline void cleanupProxy() { - if (g_proxyList()->isEmpty()) { - return; - } - const auto &data = *g_proxyList(); - qDeleteAll(data); - g_proxyList()->clear(); - } - static inline NSWindowProxy *ensureWindowProxy(const WId windowId) { auto it = g_proxyList()->find(windowId); if (it == g_proxyList()->end()) { @@ -231,12 +217,14 @@ const auto proxy = new NSWindowProxy(nswindow); it = g_proxyList()->insert(windowId, proxy); } - static bool cleanerInstalled = false; - if (!cleanerInstalled) { - cleanerInstalled = true; - qAddPostRoutine(cleanupProxy); - } return it.value(); + } + + static inline void releaseWindowProxy(const WId windowId) { + if (const auto proxy = g_proxyList()->take(windowId)) { + proxy->setSystemTitleBarVisible(true); + delete proxy; + } } class CocoaWindowEventFilter : public QObject { @@ -267,22 +255,20 @@ CocoaWindowEventFilter::~CocoaWindowEventFilter() = default; - bool CocoaWindowEventFilter::eventFilter(QObject *object, QEvent *event) { + bool CocoaWindowEventFilter::eventFilter(QObject *obj, QEvent *event) { + Q_UNUSED(obj) auto type = event->type(); if (type < QEvent::MouseButtonPress || type > QEvent::MouseMove) { return false; } - QObject *host = m_context->host(); - QWindow *window = m_context->window(); - WindowItemDelegate *delegate = m_context->delegate(); + auto host = m_context->host(); + auto window = m_context->window(); + auto delegate = m_context->delegate(); auto me = static_cast<const QMouseEvent *>(event); -#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0) - QPoint scenePos = mouseEvent->scenePosition().toPoint(); - QPoint globalPos = mouseEvent->globalPosition().toPoint(); -#else - QPoint scenePos = me->windowPos().toPoint(); - QPoint globalPos = me->screenPos().toPoint(); -#endif + + QPoint scenePos = getMouseEventScenePos(me); + QPoint globalPos = getMouseEventGlobalPos(me); + bool inTitleBar = m_context->isInTitleBarDraggableArea(scenePos); switch (type) { case QEvent::MouseButtonPress: { @@ -296,7 +282,6 @@ event->accept(); return true; } - m_windowStatus = WaitingRelease; break; } case Qt::RightButton: { @@ -306,6 +291,7 @@ default: break; } + m_windowStatus = WaitingRelease; break; } @@ -339,7 +325,7 @@ } case PreparingMove: { m_windowStatus = Moving; - window->startSystemMove(); + startSystemMove(window); event->accept(); return true; } @@ -376,7 +362,7 @@ } CocoaWindowContext::~CocoaWindowContext() { - // TODO: deref something? + releaseWindowProxy(windowId); } QString CocoaWindowContext::key() const { @@ -384,16 +370,47 @@ } void CocoaWindowContext::virtual_hook(int id, void *data) { + switch (id) { + case SystemButtonAreaChangedHook: { + // TODO: mac system button rect updated + return; + } + + default: + break; + } + AbstractWindowContext::virtual_hook(id, data); } - bool CocoaWindowContext::setupHost() { + void CocoaWindowContext::winIdChanged() { + // If the original window id is valid, remove all resources related + if (windowId) { + releaseWindowProxy(windowId); + windowId = 0; + cocoaWindowEventFilter.reset(); + } + + if (!m_windowHandle) { + return; + } + + // Allocate new resources windowId = m_windowHandle->winId(); -#if QT_VERSION < QT_VERSION_CHECK(6, 0, 0) - m_windowHandle->setProperty("_q_mac_wantsLayer", 1); -#endif ensureWindowProxy(windowId)->setSystemTitleBarVisible(false); - std::ignore = new CocoaWindowEventFilter(this, this); - return true; + cocoaWindowEventFilter = std::make_unique<CocoaWindowEventFilter>(this, this); + } + + bool CocoaWindowContext::windowAttributeChanged(const QString &key, const QVariant &attribute, + const QVariant &oldAttribute) { + if (key == QStringLiteral("no-system-buttons")) { + if (attribute.toBool()) { + // TODO: set off + } else { + // TODO: set on + } + return true; + } + return false; } } -- Gitblit v1.9.1