[Differential] [Request, 6 lines] D4280: [Task Manager] Hide group dialog when opening applets settings

[Kai Uwe Broulik]

broulik created this revision.
broulik added reviewers: Plasma, hein.
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
  When moving focus from one Plasma window to another (e.g. in this case opening the applet config dialog), Plasma Dialog stays open even with hideOnWindowDeactivate.
  
  Since changing task manager settings might affect grouping and triggers unintentional behavior when the model is used in a GroupDialog without grouping / fildering, just hide the popup in this case.

TEST PLAN
  This also makes it consistent with Plasma which explicitly collapses applets prior to showing their config interface. System tray does the same.
  
  Opened group dialog, clicked "configure task manager", group dialog closed.
  
  Before that I could open the group dialog, then disable grouping in popups and ended up with a huge dialog of all windows that I couldn't get rid of without restarting plasma.

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  applets/taskmanager/package/contents/ui/main.qml

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

To: broulik, #plasma, hein
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/20170125/11a016ab/attachment.html>