From ba47ff741a61d3d3f28f4d97a2770224342ca5b7 Mon Sep 17 00:00:00 2001 From: Alejandro Martinez Date: Fri, 22 Mar 2024 18:05:54 -0300 Subject: [PATCH] Remove special case for Sidebar background color Intended for macOS Yosemite it has a bug which doesn't allow the Sidebar background color be changed on macOS Sonoma and may be other versions, since the forced color is now default for macOS and Yosemite has been phased out long time ago, let's remove this. Fixes #4459 --- src/Gui/Colors.cpp | 20 -------------------- 1 file changed, 20 deletions(-) diff --git a/src/Gui/Colors.cpp b/src/Gui/Colors.cpp index e49abccbc..b5044f9b3 100644 --- a/src/Gui/Colors.cpp +++ b/src/Gui/Colors.cpp @@ -467,13 +467,6 @@ GCColor::readConfig() } } } -#ifdef Q_OS_MAC - // if on yosemite set default chrome to #e5e5e5 - // TODO dgr - if (QSysInfo::productVersion().toFloat() >= 12 | QSysInfo::productVersion().toFloat() < 13) { - ColorList[CCHROME].color = QColor(0xe5,0xe5,0xe5); - } -#endif } QColor @@ -1048,19 +1041,6 @@ GCColor::applyTheme(int index) appsettings->setValue(ColorList[i].setting, colorstring); } -#ifdef Q_OS_MAC - // if on yosemite we always set default chrome to #e5e5e5 and flat - // TODO dgr - qDebug() << "productVersion:" << QSysInfo::productVersion(); - if (QSysInfo::productVersion().toFloat() >= 12 | QSysInfo::productVersion().toFloat() < 13) { - QColor color = QColor(0xe5,0xe5,0xe5); - ColorList[CCHROME].color = color; - QString colorstring = QString("%1:%2:%3").arg(color.red()) - .arg(color.green()) - .arg(color.blue()); - appsettings->setValue(ColorList[CCHROME].setting, colorstring); - } -#endif } //