use Fusion qml style
authorMatthieu Gallien <matthieu.gallien@nextcloud.com>
Wed, 10 May 2023 10:10:44 +0000 (12:10 +0200)
committerMatthieu Gallien <matthieu.gallien@nextcloud.com>
Wed, 17 May 2023 16:16:09 +0000 (18:16 +0200)
allows use of more standard qml styling and components

Signed-off-by: Matthieu Gallien <matthieu.gallien@nextcloud.com>
src/gui/main.cpp

index a53aa160b1d54322671713098e69f9f3d1e1b5f8..6742a8b1a0e5e708e97868cf7fddbcef33ae6a87 100644 (file)
@@ -61,13 +61,6 @@ int main(int argc, char **argv)
     Q_INIT_RESOURCE(resources);
     Q_INIT_RESOURCE(theme);
 
-    // Work around a bug in KDE's qqc2-desktop-style which breaks
-    // buttons with icons not based on a name, by forcing a style name
-    // the platformtheme plugin won't try to force qqc2-desktops-style
-    // anymore.
-    // Can be removed once the bug in qqc2-desktop-style is gone.
-    QQuickStyle::setStyle("Fusion");
-
     // OpenSSL 1.1.0: No explicit initialisation or de-initialisation is necessary.
 
     QCoreApplication::setAttribute(Qt::AA_UseHighDpiPixmaps, true);
@@ -80,6 +73,9 @@ int main(int argc, char **argv)
     surfaceFormat.setOption(QSurfaceFormat::ResetNotification);
     QSurfaceFormat::setDefaultFormat(surfaceFormat);
 
+    QQuickWindow::setTextRenderType(QQuickWindow::NativeTextRendering);
+    QQuickStyle::setStyle(QStringLiteral("Fusion"));
+
     OCC::Application app(argc, argv);
 
 #ifdef Q_OS_WIN
@@ -104,15 +100,6 @@ int main(int argc, char **argv)
         return 0;
     }
 
-#if QT_VERSION >= QT_VERSION_CHECK(5, 14, 0)
-    QQuickWindow::setTextRenderType(QQuickWindow::NativeTextRendering);
-#else
-    // See https://bugreports.qt.io/browse/QTBUG-70481
-    if (std::fmod(app.devicePixelRatio(), 1) == 0) {
-        QQuickWindow::setTextRenderType(QQuickWindow::NativeTextRendering);
-    }
-#endif
-
 // check a environment variable for core dumps
 #ifdef Q_OS_UNIX
     if (!qEnvironmentVariableIsEmpty("OWNCLOUD_CORE_DUMP")) {