From 469c975a708ed86c834b59ed751a4548c693a7b9 Mon Sep 17 00:00:00 2001
From: 李威 wei.li <108964761+liwey1985@users.noreply.github.com>
Date: 周一, 21 4月 2025 15:34:54 +0800
Subject: [PATCH] Fix NSWindowProxy::setSystemButtonVisible not taking effect (#175)
---
src/core/contexts/cocoawindowcontext.mm | 407 ++++++++++++++++++++++++++++++++++++++++++++++++----------
1 files changed, 336 insertions(+), 71 deletions(-)
diff --git a/src/core/contexts/cocoawindowcontext.mm b/src/core/contexts/cocoawindowcontext.mm
index 21616d5..28738c6 100644
--- a/src/core/contexts/cocoawindowcontext.mm
+++ b/src/core/contexts/cocoawindowcontext.mm
@@ -1,3 +1,7 @@
+// 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 "cocoawindowcontext_p.h"
#include <objc/runtime.h>
@@ -10,26 +14,29 @@
#include "qwkglobal_p.h"
#include "systemwindow_p.h"
+// https://forgetsou.github.io/2020/11/06/macos%E5%BC%80%E5%8F%91-%E5%85%B3%E9%97%AD-%E6%9C%80%E5%B0%8F%E5%8C%96-%E5%85%A8%E5%B1%8F%E5%B1%85%E4%B8%AD%E5%A4%84%E7%90%86(%E4%BB%BFMac%20QQ)/
+// https://nyrra33.com/2019/03/26/changing-titlebars-height/
+
namespace QWK {
struct NSWindowProxy;
using ProxyList = QHash<WId, NSWindowProxy *>;
Q_GLOBAL_STATIC(ProxyList, g_proxyList);
-
- using ProxyList2 = QHash<NSWindow *, NSWindowProxy *>;
- Q_GLOBAL_STATIC(ProxyList2, g_proxyIndexes);
-
}
struct QWK_NSWindowDelegate {
public:
+ enum NSEventType {
+ WillEnterFullScreen,
+ DidEnterFullScreen,
+ WillExitFullScreen,
+ DidExitFullScreen,
+ DidResize,
+ };
+
virtual ~QWK_NSWindowDelegate() = default;
- virtual void windowWillEnterFullScreen(){};
- virtual void windowDidEnterFullScreen(){};
- virtual void windowWillExitFullScreen(){};
- virtual void windowDidExitFullScreen(){};
- virtual void windowDidResize(){};
+ virtual void windowEvent(NSEventType eventType) = 0;
};
//
@@ -76,39 +83,53 @@
- (void)windowWillEnterFullScreen:(NSNotification *)notification {
auto nswindow = reinterpret_cast<NSWindow *>(notification.object);
- if (auto proxy = QWK::g_proxyIndexes->value(nswindow)) {
- reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowWillEnterFullScreen();
+ auto nsview = [nswindow contentView];
+ if (auto proxy = QWK::g_proxyList->value(reinterpret_cast<WId>(nsview))) {
+ reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowEvent(
+ QWK_NSWindowDelegate::WillEnterFullScreen);
}
}
- (void)windowDidEnterFullScreen:(NSNotification *)notification {
auto nswindow = reinterpret_cast<NSWindow *>(notification.object);
- if (auto proxy = QWK::g_proxyIndexes->value(nswindow)) {
- reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowDidEnterFullScreen();
+ auto nsview = [nswindow contentView];
+ if (auto proxy = QWK::g_proxyList->value(reinterpret_cast<WId>(nsview))) {
+ reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowEvent(
+ QWK_NSWindowDelegate::DidEnterFullScreen);
}
}
- (void)windowWillExitFullScreen:(NSNotification *)notification {
auto nswindow = reinterpret_cast<NSWindow *>(notification.object);
- if (auto proxy = QWK::g_proxyIndexes->value(nswindow)) {
- reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowWillExitFullScreen();
+ auto nsview = [nswindow contentView];
+ if (auto proxy = QWK::g_proxyList->value(reinterpret_cast<WId>(nsview))) {
+ reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowEvent(
+ QWK_NSWindowDelegate::WillExitFullScreen);
}
}
- (void)windowDidExitFullScreen:(NSNotification *)notification {
auto nswindow = reinterpret_cast<NSWindow *>(notification.object);
- if (auto proxy = QWK::g_proxyIndexes->value(nswindow)) {
- reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowDidExitFullScreen();
+ auto nsview = [nswindow contentView];
+ if (auto proxy = QWK::g_proxyList->value(reinterpret_cast<WId>(nsview))) {
+ reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowEvent(
+ QWK_NSWindowDelegate::DidExitFullScreen);
}
}
- (void)windowDidResize:(NSNotification *)notification {
auto nswindow = reinterpret_cast<NSWindow *>(notification.object);
- if (auto proxy = QWK::g_proxyIndexes->value(nswindow)) {
- reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowDidResize();
+ auto nsview = [nswindow contentView];
+ if (auto proxy = QWK::g_proxyList->value(reinterpret_cast<WId>(nsview))) {
+ reinterpret_cast<QWK_NSWindowDelegate *>(proxy)->windowEvent(
+ QWK_NSWindowDelegate::DidResize);
}
}
+@end
+
+@interface QWK_NSViewObserver : NSObject
+- (instancetype)initWithProxy:(QWK::NSWindowProxy*)proxy;
@end
//
@@ -118,38 +139,200 @@
namespace QWK {
struct NSWindowProxy : public QWK_NSWindowDelegate {
- NSWindowProxy(NSWindow *macWindow) {
- nswindow = macWindow;
- g_proxyIndexes->insert(nswindow, this);
+ enum class BlurMode {
+ Dark,
+ Light,
+ None,
+ };
+
+ NSWindowProxy(NSView *macView) {
+ nsview = macView;
+
+ observer = [[QWK_NSViewObserver alloc] initWithProxy:this];
+ [nsview addObserver:observer
+ forKeyPath:@"window"
+ options:NSKeyValueObservingOptionNew | NSKeyValueObservingOptionOld
+ context:nil];
}
~NSWindowProxy() override {
- g_proxyIndexes->remove(nswindow);
+ [nsview removeObserver:observer forKeyPath:@"window"];
+ [observer release];
}
- void windowWillEnterFullScreen() override {
- qDebug() << "windowWillEnterFullScreen";
+ // Delegate
+ void windowEvent(NSEventType eventType) override {
+ switch (eventType) {
+ case WillExitFullScreen: {
+ if (!screenRectCallback || !systemButtonVisible)
+ return;
+
+ // The system buttons will stuck at their default positions when the
+ // exit-fullscreen animation is running, we need to hide them until the
+ // animation finishes
+ for (const auto &button : systemButtons()) {
+ button.hidden = true;
+ }
+ break;
+ }
+
+ case DidExitFullScreen: {
+ if (!screenRectCallback || !systemButtonVisible)
+ return;
+
+ for (const auto &button : systemButtons()) {
+ button.hidden = false;
+ }
+ updateSystemButtonRect();
+ break;
+ }
+
+ case DidResize: {
+ if (!screenRectCallback || !systemButtonVisible) {
+ return;
+ }
+ updateSystemButtonRect();
+ break;
+ }
+
+ default:
+ break;
+ }
}
- void windowDidEnterFullScreen() override {
- qDebug() << "windowDidEnterFullScreen";
+ // System buttons visibility
+ void setSystemButtonVisible(bool visible) {
+ systemButtonVisible = visible;
+ for (const auto &button : systemButtons()) {
+ button.hidden = !visible;
+ }
+
+ if (!screenRectCallback || !visible) {
+ return;
+ }
+ updateSystemButtonRect();
}
- void windowWillExitFullScreen() override {
- qDebug() << "windowWillExitFullScreen";
+ // System buttons area
+ void setScreenRectCallback(const ScreenRectCallback &callback) {
+ screenRectCallback = callback;
+
+ if (!callback || !systemButtonVisible) {
+ return;
+ }
+ updateSystemButtonRect();
}
- void windowDidExitFullScreen() override {
- qDebug() << "windowDidExitFullScreen";
+ void updateSystemButtonRect() {
+ if (!screenRectCallback || !systemButtonVisible) {
+ return;
+ }
+ const auto &buttons = systemButtons();
+ const auto &leftButton = buttons[0];
+ const auto &midButton = buttons[1];
+ const auto &rightButton = buttons[2];
+
+ auto titlebar = rightButton.superview;
+ int titlebarHeight = titlebar.frame.size.height;
+
+ auto spacing = midButton.frame.origin.x - leftButton.frame.origin.x;
+ auto width = midButton.frame.size.width;
+ auto height = midButton.frame.size.height;
+
+ auto viewSize = nsview.frame.size;
+ QPoint center = screenRectCallback(QSize(viewSize.width, titlebarHeight)).center();
+
+ // The origin of the NSWindow coordinate system is in the lower left corner, we
+ // do the necessary transformations
+ center.ry() = titlebarHeight - center.y();
+
+ // Mid button
+ NSPoint centerOrigin = {
+ center.x() - width / 2,
+ center.y() - height / 2,
+ };
+ [midButton setFrameOrigin:centerOrigin];
+
+ // Left button
+ NSPoint leftOrigin = {
+ centerOrigin.x - spacing,
+ centerOrigin.y,
+ };
+ [leftButton setFrameOrigin:leftOrigin];
+
+ // Right button
+ NSPoint rightOrigin = {
+ centerOrigin.x + spacing,
+ centerOrigin.y,
+ };
+ [rightButton setFrameOrigin:rightOrigin];
}
- void windowDidResize() override {
- qDebug() << "windowDidResize";
+ inline std::array<NSButton *, 3> systemButtons() {
+ auto nswindow = [nsview window];
+ if (!nswindow) {
+ return {nullptr, nullptr, nullptr};
+ }
+ NSButton *closeBtn = [nswindow standardWindowButton:NSWindowCloseButton];
+ NSButton *minimizeBtn = [nswindow standardWindowButton:NSWindowMiniaturizeButton];
+ NSButton *zoomBtn = [nswindow standardWindowButton:NSWindowZoomButton];
+ return {closeBtn, minimizeBtn, zoomBtn};
}
+ inline int titleBarHeight() const {
+ auto nswindow = [nsview window];
+ if (!nswindow) {
+ return 0;
+ }
+ NSButton *closeBtn = [nswindow standardWindowButton:NSWindowCloseButton];
+ return closeBtn.superview.frame.size.height;
+ }
+
+ // Blur effect
+ bool setBlurEffect(BlurMode mode) {
+ static Class visualEffectViewClass = NSClassFromString(@"NSVisualEffectView");
+ if (!visualEffectViewClass)
+ return false;
+
+ NSVisualEffectView *effectView = nil;
+ for (NSView *subview in [[nsview superview] subviews]) {
+ if ([subview isKindOfClass:visualEffectViewClass]) {
+ effectView = reinterpret_cast<NSVisualEffectView *>(subview);
+ }
+ }
+ if (effectView == nil) {
+ return false;
+ }
+
+ static const auto originalMaterial = effectView.material;
+ static const auto originalBlendingMode = effectView.blendingMode;
+ static const auto originalState = effectView.state;
+
+ if (mode == BlurMode::None) {
+ effectView.material = originalMaterial;
+ effectView.blendingMode = originalBlendingMode;
+ effectView.state = originalState;
+ effectView.appearance = nil;
+ } else {
+ effectView.material = NSVisualEffectMaterialUnderWindowBackground;
+ effectView.blendingMode = NSVisualEffectBlendingModeBehindWindow;
+ effectView.state = NSVisualEffectStateFollowsWindowActiveState;
+
+ if (mode == BlurMode::Dark) {
+ effectView.appearance =
+ [NSAppearance appearanceNamed:@"NSAppearanceNameVibrantDark"];
+ } else {
+ effectView.appearance =
+ [NSAppearance appearanceNamed:@"NSAppearanceNameVibrantLight"];
+ }
+ }
+ return true;
+ }
+
+ // System title bar
void setSystemTitleBarVisible(const bool visible) {
- NSView *nsview = [nswindow contentView];
- if (!nsview) {
+ auto nswindow = [nsview window];
+ if (!nswindow) {
return;
}
@@ -163,17 +346,13 @@
nswindow.titlebarAppearsTransparent = (visible ? NO : YES);
nswindow.titleVisibility = (visible ? NSWindowTitleVisible : NSWindowTitleHidden);
nswindow.hasShadow = YES;
- nswindow.showsToolbarButton = NO;
+ // nswindow.showsToolbarButton = NO;
nswindow.movableByWindowBackground = NO;
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))
- [nswindow standardWindowButton:NSWindowCloseButton].hidden = (visible ? NO : YES);
- [nswindow standardWindowButton:NSWindowMiniaturizeButton].hidden = (visible ? NO : YES);
- [nswindow standardWindowButton:NSWindowZoomButton].hidden = (visible ? NO : YES);
-#endif
+ [nswindow standardWindowButton:NSWindowCloseButton].hidden = NO;
+ [nswindow standardWindowButton:NSWindowMiniaturizeButton].hidden = NO;
+ [nswindow standardWindowButton:NSWindowZoomButton].hidden = NO;
}
static void replaceImplementations() {
@@ -233,9 +412,13 @@
windowObserver = nil;
}
- private:
+ static inline const Class windowClass = [NSWindow class];
+
+ protected:
static BOOL canBecomeKeyWindow(id obj, SEL sel) {
- if (g_proxyIndexes->contains(reinterpret_cast<NSWindow *>(obj))) {
+ auto nswindow = reinterpret_cast<NSWindow *>(obj);
+ auto nsview = [nswindow contentView];
+ if (g_proxyList->contains(reinterpret_cast<WId>(nsview))) {
return YES;
}
@@ -247,7 +430,9 @@
}
static BOOL canBecomeMainWindow(id obj, SEL sel) {
- if (g_proxyIndexes->contains(reinterpret_cast<NSWindow *>(obj))) {
+ auto nswindow = reinterpret_cast<NSWindow *>(obj);
+ auto nsview = [nswindow contentView];
+ if (g_proxyList->contains(reinterpret_cast<WId>(nsview))) {
return YES;
}
@@ -259,7 +444,9 @@
}
static void setStyleMask(id obj, SEL sel, NSWindowStyleMask styleMask) {
- if (g_proxyIndexes->contains(reinterpret_cast<NSWindow *>(obj))) {
+ auto nswindow = reinterpret_cast<NSWindow *>(obj);
+ auto nsview = [nswindow contentView];
+ if (g_proxyList->contains(reinterpret_cast<WId>(nsview))) {
styleMask |= NSWindowStyleMaskFullSizeContentView;
}
@@ -269,7 +456,9 @@
}
static void setTitlebarAppearsTransparent(id obj, SEL sel, BOOL transparent) {
- if (g_proxyIndexes->contains(reinterpret_cast<NSWindow *>(obj))) {
+ auto nswindow = reinterpret_cast<NSWindow *>(obj);
+ auto nsview = [nswindow contentView];
+ if (g_proxyList->contains(reinterpret_cast<WId>(nsview))) {
transparent = YES;
}
@@ -299,12 +488,14 @@
#endif
}
- static inline const Class windowClass = [NSWindow class];
-
private:
Q_DISABLE_COPY(NSWindowProxy)
- NSWindow *nswindow = nil;
+ NSView *nsview = nil;
+ QWK_NSViewObserver* observer = nil;
+
+ bool systemButtonVisible = true;
+ ScreenRectCallback screenRectCallback;
static inline QWK_NSWindowObserver *windowObserver = nil;
@@ -338,8 +529,8 @@
auto it = g_proxyList->find(windowId);
if (it == g_proxyList->end()) {
- NSWindow *nswindow = mac_getNSWindow(windowId);
- const auto proxy = new NSWindowProxy(nswindow);
+ NSView *nsview = reinterpret_cast<NSView *>(windowId);
+ const auto proxy = new NSWindowProxy(nsview);
it = g_proxyList->insert(windowId, proxy);
}
return it.value();
@@ -347,7 +538,10 @@
static inline void releaseWindowProxy(const WId windowId) {
if (auto proxy = g_proxyList->take(windowId)) {
- proxy->setSystemTitleBarVisible(true);
+ // TODO: Determine if the window is valid
+
+ // The window has been destroyed
+ // proxy->setSystemTitleBarVisible(true);
delete proxy;
} else {
return;
@@ -468,10 +662,11 @@
}
case QEvent::MouseButtonDblClick: {
- if (me->button() == Qt::LeftButton && inTitleBar &&
- !delegate->isHostSizeFixed(host)) {
+ if (me->button() == Qt::LeftButton && inTitleBar && !m_context->isHostSizeFixed()) {
+ Qt::WindowFlags windowFlags = delegate->getWindowFlags(host);
Qt::WindowStates windowState = delegate->getWindowState(host);
- if (!(windowState & Qt::WindowFullScreen)) {
+ if ((windowFlags & Qt::WindowMaximizeButtonHint) &&
+ !(windowState & Qt::WindowFullScreen)) {
if (windowState & Qt::WindowMaximized) {
delegate->setWindowState(host, windowState & ~Qt::WindowMaximized);
} else {
@@ -495,7 +690,7 @@
}
CocoaWindowContext::~CocoaWindowContext() {
- releaseWindowProxy(windowId);
+ releaseWindowProxy(m_windowId);
}
QString CocoaWindowContext::key() const {
@@ -505,7 +700,7 @@
void CocoaWindowContext::virtual_hook(int id, void *data) {
switch (id) {
case SystemButtonAreaChangedHook: {
- // TODO: mac system button rect updated
+ ensureWindowProxy(m_windowId)->setScreenRectCallback(m_systemButtonAreaCallback);
return;
}
@@ -515,33 +710,103 @@
AbstractWindowContext::virtual_hook(id, data);
}
- void CocoaWindowContext::winIdChanged() {
+ QVariant CocoaWindowContext::windowAttribute(const QString &key) const {
+ if (key == QStringLiteral("title-bar-height")) {
+ if (!m_windowId)
+ return {};
+ return ensureWindowProxy(m_windowId)->titleBarHeight();
+ }
+ return AbstractWindowContext::windowAttribute(key);
+ }
+
+ void CocoaWindowContext::winIdChanged(WId winId, WId oldWinId) {
// If the original window id is valid, remove all resources related
- if (windowId) {
- releaseWindowProxy(windowId);
- windowId = 0;
+ if (oldWinId) {
+ releaseWindowProxy(oldWinId);
}
- if (!m_windowHandle) {
+ if (!winId) {
return;
}
// Allocate new resources
- windowId = m_windowHandle->winId();
- ensureWindowProxy(windowId)->setSystemTitleBarVisible(false);
+ const auto proxy = ensureWindowProxy(winId);
+ if (proxy) {
+ proxy->setSystemButtonVisible(!windowAttribute(QStringLiteral("no-system-buttons")).toBool());
+ proxy->setScreenRectCallback(m_systemButtonAreaCallback);
+ proxy->setSystemTitleBarVisible(false);
+ }
}
bool CocoaWindowContext::windowAttributeChanged(const QString &key, const QVariant &attribute,
const QVariant &oldAttribute) {
+ Q_UNUSED(oldAttribute)
+
+ Q_ASSERT(m_windowId);
+
if (key == QStringLiteral("no-system-buttons")) {
- if (attribute.toBool()) {
- // TODO: set off
- } else {
- // TODO: set on
- }
+ if (attribute.type() != QVariant::Bool)
+ return false;
+ ensureWindowProxy(m_windowId)->setSystemButtonVisible(!attribute.toBool());
return true;
+ }
+
+ if (key == QStringLiteral("blur-effect")) {
+ auto mode = NSWindowProxy::BlurMode::None;
+ if (attribute.type() == QVariant::Bool) {
+ if (attribute.toBool()) {
+ NSString *osxMode =
+ [[NSUserDefaults standardUserDefaults] stringForKey:@"AppleInterfaceStyle"];
+ mode = [osxMode isEqualToString:@"Dark"] ? NSWindowProxy::BlurMode::Dark
+ : NSWindowProxy::BlurMode::Light;
+ }
+ } else if (attribute.type() == QVariant::String) {
+ auto value = attribute.toString();
+ if (value == QStringLiteral("dark")) {
+ mode = NSWindowProxy::BlurMode::Dark;
+ } else if (value == QStringLiteral("light")) {
+ mode = NSWindowProxy::BlurMode::Light;
+ } else if (value == QStringLiteral("none")) {
+ // ...
+ } else {
+ return false;
+ }
+ } else {
+ return false;
+ }
+ return ensureWindowProxy(m_windowId)->setBlurEffect(mode);
}
return false;
}
}
+
+@implementation QWK_NSViewObserver {
+ QWK::NSWindowProxy* _proxy; // Weak reference
+}
+
+- (instancetype)initWithProxy:(QWK::NSWindowProxy*)proxy {
+ if (self = [super init]) {
+ _proxy = proxy;
+ }
+ return self;
+}
+
+// Using QEvent::Show to call setSystemTitleBarVisible/updateSystemButtonRect could also work,
+// but observing the window property change via KVO provides more immediate notification when
+// the NSWindow becomes available, making this approach more natural and reliable.
+- (void)observeValueForKeyPath:(NSString*)keyPath
+ ofObject:(id)object
+ change:(NSDictionary*)change
+ context:(void*)context {
+ if ([keyPath isEqualToString:@"window"]) {
+ NSWindow* newWindow = change[NSKeyValueChangeNewKey];
+ // NSWindow* oldWindow = change[NSKeyValueChangeOldKey];
+ if (newWindow) {
+ _proxy->setSystemTitleBarVisible(false);
+ _proxy->updateSystemButtonRect();
+ }
+ }
+}
+
+@end
--
Gitblit v1.9.1