[Differential] [Commented On] D2345: use a separate configuration per look and feel

[mart (Marco Martin)]

mart added inline comments.

INLINE COMMENTS

> davidedmundson wrote in shellcorona.cpp:383
> No!!!!
> 
> you need to loop through containments() rather than the views
> (same for panel section)
> 
> Otherwise:
> 
> - you're only saving the currently plugged in screens.
> - this won't save the configuration for any applets in a system tray.
> 
> and when you do do the system tray you're going to have a huge problem:
> system tray writes out SystrayContainmentId... you aren't going to know what that is.

can't for panels, as their geometry is saved in the view as it should be, because they have a separate geometry setting per screen.

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

To: mart, davidedmundson, #plasma
Cc: davidedmundson, ivan, plasma-devel, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160809/3e9215dc/attachment-0001.html>