Backported lxqtplatformtheme-follow-color-scheme.patch

* Fixed wrong group in VCS fields
* Removed some trailing whitespaces from d/changelog

debian/changelog

@@ -1,3 +1,11 @@
+lxqt-qtplugin (0.12.0-6) unstable; urgency=medium
+
+  * Backported lxqtplatformtheme-follow-color-scheme.patch
+  * Fixed wrong group in VCS fields
+  * Removed some trailing whitespaces from d/changelog
+
+ -- Alf Gaida <agaida@siduction.org>  Wed, 14 Feb 2018 22:42:02 +0100
+
 lxqt-qtplugin (0.12.0-5) unstable; urgency=medium
 
   * Backported colorize-at-start patch
@@ -13,13 +21,13 @@ lxqt-qtplugin (0.12.0-4) unstable; urgency=medium
 
   * Bumped Standards to 4.1.2, no changes needed
   * Removed branch from VCS fields
-  * Removed debian/gbp.conf 
+  * Removed debian/gbp.conf
 
  -- Alf Gaida <agaida@siduction.org>  Thu, 14 Dec 2017 22:31:53 +0100
 
 lxqt-qtplugin (0.12.0-3) unstable; urgency=medium
 
-  * Transition to unstable 
+  * Transition to unstable
 
  -- Alf Gaida <agaida@siduction.org>  Tue, 05 Dec 2017 00:18:17 +0100
 
@@ -80,7 +88,7 @@ lxqt-qtplugin (0.11.0-3) unstable; urgency=medium
   * Bumped minimum version libqtxdg-dev (>= 2.0.0)
   * Added build dependency libqt5svg5-dev
   * Added build dependency libqt5xdgiconloader-dev
-  * Fixed VCS fields, using https and plain /git/ 
+  * Fixed VCS fields, using https and plain /git/
   * Removed README.debian, no longer needed
   * Fixed copyrights Format field to https
   * Bumped years in copyrights
@@ -91,12 +99,12 @@ lxqt-qtplugin (0.11.0-3) unstable; urgency=medium
   * Set CMAKE_BUILD_TYPE=RelWithDebInfo
   * Exported LC_ALL=C.UTF-8, make builds reproducible
   * Added a basic gbp.conf
- 
+
  -- Alf Gaida <agaida@siduction.org>  Tue, 18 Oct 2016 17:44:28 +0200
 
 lxqt-qtplugin (0.10.0-3) unstable; urgency=medium
 
-  * Remove dbg package in favor of dbgsym. 
+  * Remove dbg package in favor of dbgsym.
 
  -- Alf Gaida <agaida@siduction.org>  Sat, 26 Dec 2015 22:27:15 +0100
 

debian/control

@@ -15,8 +15,8 @@ Build-Depends: debhelper (>= 11),
                lxqt-build-tools (>= 0.4.0),
                qtbase5-private-dev
 Standards-Version: 4.1.3
-Vcs-Browser: https://salsa.debian.org/team-lxqt/lxqt-qtplugin.git
+Vcs-Browser: https://salsa.debian.org/lxqt-team/lxqt-qtplugin
-Vcs-Git: https://salsa.debian.org/team-lxqt/lxqt-qtplugin.git
+Vcs-Git: https://salsa.debian.org/lxqt-team/lxqt-qtplugin
 Homepage: https://github.com/lxde/lxqt-qtplugin
 
 Package: lxqt-qtplugin

debian/patches/lxqtplatformtheme-follow-color-scheme.patch

@@ -0,0 +1,80 @@
+From c78433c32c327d09507976a7b5cecff87c224252 Mon Sep 17 00:00:00 2001
+From: Palo Kisa <palo.kisa@gmail.com>
+Date: Tue, 13 Feb 2018 10:01:10 +0100
+Subject: [PATCH] lxqtplatformtheme: Initialize "folowColorScheme" once
+
+---
+ src/lxqtplatformtheme.cpp | 18 +++++++++++++-----
+ src/lxqtplatformtheme.h   |  4 +++-
+ 2 files changed, 16 insertions(+), 6 deletions(-)
+
+diff --git a/src/lxqtplatformtheme.cpp b/src/lxqtplatformtheme.cpp
+index 8291eab..467cf07 100644
+--- a/src/lxqtplatformtheme.cpp
++++ b/src/lxqtplatformtheme.cpp
+@@ -52,13 +52,20 @@ LXQtPlatformTheme::LXQtPlatformTheme():
+     iconFollowColorScheme_(true)
+     , settingsWatcher_(NULL)
+ {
+-    // When the plugin is loaded, it seems that the app is not yet running and
++    loadSettings();
++    // Note: When the plugin is loaded, it seems that the app is not yet running and
+     // QThread environment is not completely set up. So creating filesystem watcher
+     // does not work since it uses QSocketNotifier internally which can only be
+     // created within QThread thread. So let's schedule a idle handler to initialize
+     // the watcher in the main event loop.
+-    loadSettings();
+-    QTimer::singleShot(0, this, SLOT(initWatch()));
++
++    // Note2: the QTimer::singleShot with also needs a QThread environment
++    // (there is a workaround in qtcore for the 0-timer with the old SLOT notation)
++
++    // TODO: use the "PointerToMemberFunction" overloads of invokeMethod(), but they
++    // are available from Qt v5.10
++    //QMetaObject::ivokeMethod(this, &LXQtPlatformTheme::lazyInit, Qt::QueuedConnection);
++    QMetaObject::invokeMethod(this, "lazyInit", Qt::QueuedConnection);
+ }
+ 
+ LXQtPlatformTheme::~LXQtPlatformTheme() {
+@@ -66,11 +73,13 @@ LXQtPlatformTheme::~LXQtPlatformTheme() {
+         delete settingsWatcher_;
+ }
+ 
+-void LXQtPlatformTheme::initWatch()
++void LXQtPlatformTheme::lazyInit()
+ {
+     settingsWatcher_ = new QFileSystemWatcher();
+     settingsWatcher_->addPath(settingsFile_);
+     connect(settingsWatcher_, &QFileSystemWatcher::fileChanged, this, &LXQtPlatformTheme::onSettingsChanged);
++
++    XdgIconLoader::instance()->setFollowColorScheme(iconFollowColorScheme_);
+ }
+ 
+ void LXQtPlatformTheme::loadSettings() {
+@@ -315,7 +324,6 @@ QVariant LXQtPlatformTheme::themeHint(ThemeHint hint) const {
+ 
+ QIconEngine *LXQtPlatformTheme::createIconEngine(const QString &iconName) const
+ {
+-    XdgIconLoader::instance()->setFollowColorScheme(iconFollowColorScheme_);
+     return new XdgIconLoaderEngine(iconName);
+ }
+  
+diff --git a/src/lxqtplatformtheme.h b/src/lxqtplatformtheme.h
+index 3c07c46..d0e7b6e 100644
+--- a/src/lxqtplatformtheme.h
++++ b/src/lxqtplatformtheme.h
+@@ -78,11 +78,13 @@ class Q_GUI_EXPORT LXQtPlatformTheme : public QObject, public QPlatformTheme {
+ 
+     // virtual QString standardButtonText(int button) const;
+ 
++public Q_SLOTS:
++    void lazyInit();
++
+ private:
+     void loadSettings();
+ 
+ private Q_SLOTS:
+-    void initWatch();
+     void onSettingsChanged();
+ 
+ private:

debian/patches/series

@@ -1 +1,2 @@
 colorizing-at-startup.patch
+lxqtplatformtheme-follow-color-scheme.patch