10110111 / k4oxygen

A standalone variant of Oxygen theme for Qt 4,5,6, intended to look mostly like Oxygen@KDE4
GNU Lesser General Public License v2.1
4 stars 1 forks source link

Broken with dark color schemes #18

Open hazel-bunny opened 5 hours ago

hazel-bunny commented 5 hours ago

K4Oxygen with Materia Dark: Screenshot_20241019_162235

Oxygen with Materia Dark: Screenshot_20241019_162445

Operating System: Fedora Linux 40 KDE Plasma Version: 6.2.1 KDE Frameworks Version: 6.7.0 Qt Version: 6.7.2 Kernel Version: 6.11.4-201.fsync.fc40.x86_64 (64-bit) Graphics Platform: Wayland

10110111 commented 5 hours ago

Does this patch help? (Is kf6-config the right command in modern KDE?)

diff --git a/libs/imported/oxygenconfig.cpp b/libs/imported/oxygenconfig.cpp
index 734d73b..e538f5a 100644
--- a/libs/imported/oxygenconfig.cpp
+++ b/libs/imported/oxygenconfig.cpp
@@ -62,6 +62,9 @@ QStringList getConfigPaths()
     if(process.waitForFinished() && process.exitCode()==0)
         return QString::fromUtf8(process.readAllStandardOutput()).trimmed().split(':');
     process.start("kf5-config",args);
+    if(process.waitForFinished() && process.exitCode()==0)
+        return QString::fromUtf8(process.readAllStandardOutput()).trimmed().split(':');
+    process.start("kf6-config",args);
     if(process.waitForFinished() && process.exitCode()==0)
         return QString::fromUtf8(process.readAllStandardOutput()).trimmed().split(':');