[Differential] [Request, 7 lines] D4492: [KHintsSettings] Emit QGuiApplication::paletteChanged when run as QApplication

[Kai Uwe Broulik]

broulik created this revision.
broulik added a reviewer: Plasma.
broulik set the repository for this revision to R135 Integration for Qt applications in Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  QGuiApplication::setPalette and QApplication::setPalette are two distinct static functions and when the paletteChanged signal was introduced in QGuiApplication the latter was never adjusted.
  SystemPalette connects to this signal to avoid installing an eventFilter on qApp for ApplicationPaletteChanged.
  This causes e.g. QML-based System Settings modules to not update when changing color schemes but only the widget-based application around it.

TEST PLAN
  Switching to Breeze Dark look and feel theme now updates the background of the QML parts. There's still some glitches like QQC buttons not repainting properly but at least it's not as embarrassing to look at as it used to be.
  
  I'm trying to fix this in Qt but until this is in I think this patch is safe either way and will also benefit users of older Qt versions.
  
  https://codereview.qt-project.org/#/c/181704/

REPOSITORY
  R135 Integration for Qt applications in Plasma

REVISION DETAIL
  https://phabricator.kde.org/D4492

AFFECTED FILES
  src/platformtheme/khintssettings.cpp

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: broulik, #plasma
Cc: plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170208/f5aa8a48/attachment.html>