From 5c04046acae6e61280fba7fef31401edd7c36087 Mon Sep 17 00:00:00 2001 From: Dylan Liu <312902918@qq.com> Date: 周五, 23 5月 2025 16:04:20 +0800 Subject: [PATCH] QWKCore/cocoawindowcontext: Use typeId() instead of type() in Qt 6.0+. (#181) --- src/core/contexts/abstractwindowcontext.cpp | 195 ++++++++++++++++++++++++++++++++++++++---------- 1 files changed, 155 insertions(+), 40 deletions(-) diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp index ef1efbf..f2fc730 100644 --- a/src/core/contexts/abstractwindowcontext.cpp +++ b/src/core/contexts/abstractwindowcontext.cpp @@ -1,7 +1,12 @@ +// Copyright (C) 2023-2024 Stdware Collections (https://www.github.com/stdware) +// Copyright (C) 2021-2023 wangwenx190 (Yuhang Zhao) +// SPDX-License-Identifier: Apache-2.0 + #include "abstractwindowcontext_p.h" #include <QtGui/QPen> #include <QtGui/QPainter> +#include <QtGui/QScreen> #include "qwkglobal_p.h" @@ -17,35 +22,39 @@ } m_host = host; m_delegate.reset(delegate); - m_windowHandle = m_delegate->hostWindow(m_host); - if (m_windowHandle) { - winIdChanged(nullptr); - } + m_winIdChangeEventFilter.reset(delegate->createWinIdEventFilter(host, this)); + notifyWinIdChange(); } - bool AbstractWindowContext::setHitTestVisible(const QObject *obj, bool visible) { + bool AbstractWindowContext::setHitTestVisible(QObject *obj, bool visible) { Q_ASSERT(obj); if (!obj) { return false; } if (visible) { - m_hitTestVisibleItems.insert(obj); + m_hitTestVisibleItems.removeAll(nullptr); + m_hitTestVisibleItems.removeAll(obj); + m_hitTestVisibleItems.append(obj); } else { - m_hitTestVisibleItems.remove(obj); + for (auto &item : m_hitTestVisibleItems) { + if (item == obj) { + item = nullptr; + } + } } return true; } bool AbstractWindowContext::setSystemButton(WindowAgentBase::SystemButton button, QObject *obj) { - Q_ASSERT(obj); Q_ASSERT(button != WindowAgentBase::Unknown); - if (!obj || (button == WindowAgentBase::Unknown)) { + if (button == WindowAgentBase::Unknown) { return false; } - if (m_systemButtons[button] == obj) { + auto org = m_systemButtons[button]; + if (org == obj) { return false; } m_systemButtons[button] = obj; @@ -54,20 +63,22 @@ bool AbstractWindowContext::setTitleBar(QObject *item) { Q_ASSERT(item); - if (!item) { + auto org = m_titleBar; + if (org == item) { return false; } - if (m_titleBar == item) { - return false; + if (org) { + // Since the title bar is changed, all items inside it should be dereferenced right away + removeSystemButtonsAndHitTestItems(); } m_titleBar = item; return true; } #ifdef Q_OS_MAC - void AbstractWindowContext::setSystemButtonArea(const QRect &rect) { - m_systemButtonArea = rect; + void AbstractWindowContext::setSystemButtonAreaCallback(const ScreenRectCallback &callback) { + m_systemButtonAreaCallback = callback; virtual_hook(SystemButtonAreaChangedHook, nullptr); } #endif @@ -77,8 +88,7 @@ *button = WindowAgentBase::Unknown; for (int i = WindowAgentBase::WindowIcon; i <= WindowAgentBase::Close; ++i) { auto currentButton = m_systemButtons[i]; - if (!currentButton || !m_delegate->isVisible(currentButton) || - !m_delegate->isEnabled(currentButton)) { + if (!currentButton || !m_delegate->isVisible(currentButton)) { continue; } if (m_delegate->mapGeometryToScene(currentButton).contains(pos)) { @@ -111,22 +121,17 @@ return false; } - for (int i = WindowAgentBase::WindowIcon; i <= WindowAgentBase::Close; ++i) { - auto currentButton = m_systemButtons[i]; - if (currentButton && m_delegate->isVisible(currentButton) && - m_delegate->isEnabled(currentButton) && - m_delegate->mapGeometryToScene(currentButton).contains(pos)) { + WindowAgentBase::SystemButton button; + if (isInSystemButtons(pos, &button)) { + return false; + } + + for (auto item : m_hitTestVisibleItems) { + if (item && m_delegate->isVisible(item) && + m_delegate->mapGeometryToScene(item).contains(pos)) { return false; } } - - for (auto widget : m_hitTestVisibleItems) { - if (widget && m_delegate->isVisible(widget) && m_delegate->isEnabled(widget) && - m_delegate->mapGeometryToScene(widget).contains(pos)) { - return false; - } - } - return true; } @@ -144,13 +149,30 @@ void AbstractWindowContext::virtual_hook(int id, void *data) { switch (id) { case CentralizeHook: { - // TODO: Qt - break; + if (!m_windowId) + return; + + QRect windowGeometry = m_delegate->getGeometry(m_host); + QRect screenGeometry = m_windowHandle->screen()->geometry(); + int x = (screenGeometry.width() - windowGeometry.width()) / 2; + int y = (screenGeometry.height() - windowGeometry.height()) / 2; + QPoint pos(x, y); + pos += screenGeometry.topLeft(); + m_delegate->setGeometry(m_host, QRect(pos, windowGeometry.size())); + return; } - case ShowSystemMenuHook: { - // TODO: Qt - break; + case RaiseWindowHook: { + if (!m_windowId) + return; + + m_delegate->setWindowVisible(m_host, true); + Qt::WindowStates state = m_delegate->getWindowState(m_host); + if (state & Qt::WindowMinimized) { + m_delegate->setWindowState(m_host, state & ~Qt::WindowMinimized); + } + m_delegate->bringWindowToTop(m_host); + return; } case DefaultColorsHook: { @@ -173,11 +195,104 @@ } void AbstractWindowContext::notifyWinIdChange() { - auto oldWindow = m_windowHandle; - m_windowHandle = m_delegate->window(m_host); - if (oldWindow == m_windowHandle) - return; - winIdChanged(oldWindow); + auto oldWinId = m_windowId; + m_windowId = m_winIdChangeEventFilter->winId(); + + // In Qt6, after QWidget::close() is called, the related QWindow's all surfaces and the + // platform window will be removed, and the WinId will be set to 0. After that, when the + // QWidget is shown again, the whole things will be recreated again. + // As a result, we must update our WindowContext each time the WinId changes. + if (m_windowHandle) { + removeEventFilter(m_windowHandle); + } + m_windowHandle = m_delegate->hostWindow(m_host); + if (m_windowHandle) { + m_windowHandle->installEventFilter(this); + } + + if (oldWinId != m_windowId) { + winIdChanged(m_windowId, oldWinId); + + if (m_windowId) { + // Refresh window attributes + for (auto it = m_windowAttributesOrder.begin(); + it != m_windowAttributesOrder.end();) { + if (!windowAttributeChanged(it->first, it->second, {})) { + m_windowAttributes.remove(it->first); + it = m_windowAttributesOrder.erase(it); + continue; + } + ++it; + } + } + + // Send to shared dispatchers + QEvent e(QEvent::WinIdChange); + sharedDispatch(m_host, &e); + } + } + + QVariant AbstractWindowContext::windowAttribute(const QString &key) const { + auto it = m_windowAttributes.find(key); + if (it == m_windowAttributes.end()) { + return {}; + } + return it.value()->second; + } + + bool AbstractWindowContext::setWindowAttribute(const QString &key, const QVariant &attribute) { + auto it = m_windowAttributes.find(key); + if (it == m_windowAttributes.end()) { + if (!attribute.isValid()) { + return true; + } + if (m_windowId && !windowAttributeChanged(key, attribute, {})) { + return false; + } + m_windowAttributes.insert( + key, m_windowAttributesOrder.insert(m_windowAttributesOrder.end(), + std::make_pair(key, attribute))); + return true; + } + + auto &listIter = it.value(); + auto &oldAttr = listIter->second; + if (m_windowId && !windowAttributeChanged(key, attribute, oldAttr)) { + return false; + } + + if (attribute.isValid()) { + oldAttr = attribute; + m_windowAttributesOrder.splice(m_windowAttributesOrder.end(), m_windowAttributesOrder, + listIter); + } else { + m_windowAttributesOrder.erase(listIter); + m_windowAttributes.erase(it); + } + return true; + } + + bool AbstractWindowContext::eventFilter(QObject *obj, QEvent *event) { + if (obj == m_windowHandle && sharedDispatch(obj, event)) { + return true; + } + return QObject::eventFilter(obj, event); + } + + bool AbstractWindowContext::windowAttributeChanged(const QString &key, + const QVariant &attribute, + const QVariant &oldAttribute) { + return false; + } + + void AbstractWindowContext::removeSystemButtonsAndHitTestItems() { + for (auto &button : m_systemButtons) { + if (!button) { + continue; + } + button = nullptr; + } + m_hitTestVisibleItems.clear(); } } -- Gitblit v1.9.1