From a51b5688e1c33d3ce96b48c869603b00f908f513 Mon Sep 17 00:00:00 2001 From: Sine Striker <trueful@163.com> Date: 周三, 13 12月 2023 04:16:37 +0800 Subject: [PATCH] Make better code structure for border handlers --- src/core/contexts/abstractwindowcontext.cpp | 63 +++++++++++++------------------ 1 files changed, 27 insertions(+), 36 deletions(-) diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp index 49eb0c5..2a68081 100644 --- a/src/core/contexts/abstractwindowcontext.cpp +++ b/src/core/contexts/abstractwindowcontext.cpp @@ -7,8 +7,14 @@ namespace QWK { - AbstractWindowContext::AbstractWindowContext() { - } + QWK_USED static constexpr const struct { + const quint32 activeLight = MAKE_RGBA_COLOR(210, 233, 189, 226); + const quint32 activeDark = MAKE_RGBA_COLOR(177, 205, 190, 240); + const quint32 inactiveLight = MAKE_RGBA_COLOR(193, 195, 211, 203); + const quint32 inactiveDark = MAKE_RGBA_COLOR(240, 240, 250, 255); + } kSampleColorSet; + + AbstractWindowContext::AbstractWindowContext() = default; AbstractWindowContext::~AbstractWindowContext() = default; @@ -28,7 +34,7 @@ if (!setupHost()) { m_host = nullptr; - m_delegate = nullptr; + m_delegate.reset(); m_windowHandle = nullptr; return false; } @@ -90,10 +96,6 @@ } m_titleBar = item; return true; - } - - void AbstractWindowContext::showSystemMenu(const QPoint &pos) { - virtual_hook(ShowSystemMenuHook, &const_cast<QPoint &>(pos)); } QRegion AbstractWindowContext::hitTestShape() const { @@ -172,40 +174,25 @@ return {}; } - static constexpr struct { - const quint32 grass = MAKE_RGBA_COLOR(210, 233, 189, 226); - const quint32 pink = MAKE_RGBA_COLOR(177, 205, 190, 240); - const quint32 tile = MAKE_RGBA_COLOR(193, 195, 211, 203); - const quint32 azure = MAKE_RGBA_COLOR(240, 240, 250, 255); - } kSampleColorSet; - void AbstractWindowContext::virtual_hook(int id, void *data) { switch (id) { - case NeedsDrawBordersHook: { - auto &result = *static_cast<bool *>(data); - result = false; - return; + case CentralizeHook: { + // TODO: Qt + break; } - case DrawBordersHook: { - auto args = static_cast<void **>(data); - auto &painter = *static_cast<QPainter *>(args[0]); - const auto &rect = *static_cast<const QRect *>(args[1]); - // Top - painter.setPen(kSampleColorSet.grass); - painter.drawLine(rect.topLeft(), rect.topRight()); + case ShowSystemMenuHook: { + // TODO: Qt + break; + } - // Right - painter.setPen(kSampleColorSet.pink); - painter.drawLine(rect.topRight(), rect.bottomRight()); - - // Bottom - painter.setPen(kSampleColorSet.tile); - painter.drawLine(rect.bottomLeft(), rect.bottomRight()); - - // Right - painter.setPen(kSampleColorSet.azure); - painter.drawLine(rect.topLeft(), rect.bottomLeft()); + case DefaultColorsHook: { + auto &map = *static_cast<QMap<QString, QColor> *>(data); + map.clear(); + map.insert(QStringLiteral("activeLight"), kSampleColorSet.activeLight); + map.insert(QStringLiteral("activeDark"), kSampleColorSet.activeDark); + map.insert(QStringLiteral("inactiveLight"), kSampleColorSet.inactiveLight); + map.insert(QStringLiteral("inactiveDark"), kSampleColorSet.inactiveDark); return; } default: @@ -213,4 +200,8 @@ } } + void AbstractWindowContext::showSystemMenu(const QPoint &pos) { + virtual_hook(ShowSystemMenuHook, &const_cast<QPoint &>(pos)); + } + } \ No newline at end of file -- Gitblit v1.9.1