D7935: Set QtQuickControls theme in QPT

[David Edmundson]

davidedmundson created this revision.
davidedmundson added a reviewer: Plasma.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  Currently we set QT_QUICK_CONTROLS_STYLE in startkde.
  
  This causes problems as we're setting vars that meddle with other
  developers setups. Especially if they're using QGuiApplication which
  will instantly crash when using the desktop theme.
  
  By moving the setting inside the QPT we can:
  
  - only affect QApplications
  - only affect apps that are desktopSettingsAware
  
  If this env is already explicitly set by someone else, that has
  precedence.
  
  Also if this ever does move from being an env variable, we will probably
  want to add the code here.
  
  BUG: 384466
  BUG: 384481

TEST PLAN
  Unset env
  Ran systemsettings5 the scrollbar was the correct colour

REPOSITORY
  R135 Integration for Qt applications in Plasma

BRANCH
  master

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

AFFECTED FILES
  src/platformtheme/kdeplatformtheme.cpp
  src/platformtheme/kdeplatformtheme.h

To: davidedmundson, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170922/48b2cfde/attachment-0001.html>