[Differential] [Request, 19 lines] D4199: [Style KCM] Make sure to enable kappmenu kded module

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, ivan.
broulik set the repository for this revision to R119 Plasma Desktop.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  The old KCM enabled and disabled the kded (both autoloading and load/unloadModule) depending on global menu setting.
  Since we never had global menu in 5.x before, it would disable the menu when you hit Apply.
  This will break global menu for users without an obvious reason why.

TEST PLAN
  Unloaded global menu module, hit apply, module was loaded.
  Disabled global menu autoloading, hit apply, module autoloading was enabled and module was loaded.
  
  This obviously overwrites when users disable this service but then, the service could/should always be running. It won't register a com canonical appmenu registrar service if it's not enabled, so apps will work just fine with kappmenu service running.

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  kcms/style/kcmstyle.cpp

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

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