[Differential] [Request, 10 lines] D4611: Check the background is valid when choosing background type

[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
  I rushed in an emergency fix for 5.9.1 that added support for setting
  the background type, which was a new property someone added to the
  breeze theme in 5.9.1.
  
  My last patch doesn't support clearing the background properly.
  Effectively the old code would always set the type to "image".
  
  BUG: 376484

TEST PLAN
  Set background to image. Checked config
  Cleared background. Checked config

REPOSITORY
  R123 SDDM Configuration Panel (KCM)

BRANCH
  master

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

AFFECTED FILES
  src/themeconfig.cpp

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

To: davidedmundson, #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/20170214/cee1e66d/attachment.html>