minor tweaks

Sine Striker

2023-12-20 669ea85c57567624e687d92898266f78bb55f1fa

 src/core/contexts/abstractwindowcontext.cpp

@@ -2,6 +2,7 @@

#include <QtGui/QPen>
#include <QtGui/QPainter>
#include <QtGui/QScreen>

#include "qwkglobal_p.h"

@@ -11,27 +12,32 @@

    AbstractWindowContext::~AbstractWindowContext() = default;

    bool AbstractWindowContext::setup(QObject *host, WindowItemDelegate *delegate) {
        if (!host || !delegate) {
            return false;
    void AbstractWindowContext::setup(QObject *host, WindowItemDelegate *delegate) {
        if (m_host || !host || !delegate) {
            return;
        }

        auto windowHandle = delegate->hostWindow(host);
        if (!windowHandle) {
            return false;
        }

        m_host = host;
        m_delegate.reset(delegate);
        m_windowHandle = windowHandle;

        if (!setupHost()) {
            m_host = nullptr;
            m_delegate.reset();
            m_windowHandle = nullptr;
            return false;
        m_windowHandle = m_delegate->hostWindow(m_host);
        if (m_windowHandle) {
            winIdChanged();
        }
        return true;
    }

    void AbstractWindowContext::setWindowAttribute(const QString &key, const QVariant &var) {
        auto it = m_windowAttributes.find(key);
        if (it.value() == var)
            return;

        auto newVar = var;
        auto oldVar = it.value();
        void *args[] = {
            &const_cast<QString &>(key),
            &newVar,
            &oldVar,
        };
        virtual_hook(WindowAttributeChangedHook, args);
    }

    bool AbstractWindowContext::setHitTestVisible(const QObject *obj, bool visible) {
@@ -50,9 +56,8 @@

    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;
        }

@@ -65,10 +70,6 @@

    bool AbstractWindowContext::setTitleBar(QObject *item) {
        Q_ASSERT(item);
        if (!item) {
            return false;
        }

        if (m_titleBar == item) {
            return false;
        }
@@ -155,13 +156,25 @@
    void AbstractWindowContext::virtual_hook(int id, void *data) {
        switch (id) {
            case CentralizeHook: {
                // TODO: Qt
                break;
                if (!m_windowHandle)
                    return;

                QRect screenGeometry = m_windowHandle->screen()->geometry();
                int x = (screenGeometry.width() - m_windowHandle->width()) / 2;
                int y = (screenGeometry.height() - m_windowHandle->height()) / 2;
                QPoint pos(x, y);
                pos += screenGeometry.topLeft();
                m_windowHandle->setPosition(pos);
                return;
            }

            case ShowSystemMenuHook: {
                // TODO: Qt
                break;
            case RaiseWindowHook: {
                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: {
@@ -183,4 +196,12 @@
        virtual_hook(ShowSystemMenuHook, &const_cast<QPoint &>(pos));
    }

}
    void AbstractWindowContext::notifyWinIdChange() {
        auto oldWindow = m_windowHandle;
        m_windowHandle = m_delegate->hostWindow(m_host);
        if (oldWindow == m_windowHandle)
            return;
        winIdChanged();
    }

}