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

[davidedmundson (David Edmundson)]

davidedmundson added inline comments.

INLINE COMMENTS

> shellcorona.cpp:394
>          
>          //try to parse the items geometries
>          KConfigGroup genericConf(&contConfig, QStringLiteral("General"));

I'm confused.

Should you be getting the current from the current Plasma::Applet* instances or are you loading state from the config file?

REPOSITORY
  rPLASMAWORKSPACE Plasma Workspace

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

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

To: mart, #plasma, davidedmundson
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/20160808/a656bbbd/attachment.html>