[Differential] [Commented On] D2449: small utility to create L&F packages

mart (Marco Martin) noreply at phabricator.kde.org
Tue Aug 16 17:34:33 UTC 2016


mart added inline comments.

INLINE COMMENTS

> davidedmundson wrote in lnflogic.cpp:155
> Edit: This kwin saving stuff is *still* wrong.
> 
> In systemCG (kwinrc) it is "TabBox".
> However, in the LnF it is "WindowSwitcher."
> 
> You "fixed" the wrong one.
> 
> You are also missing DesktopSwitcher.
> 
> I'll just write the code for you:
> 
>   systemCG = KConfigGroup(KSharedConfig::openConfig(QStringLiteral("kwinrc")), "TabBox");
>   defaultsConfigGroup = KConfigGroup(&defaultsConfigGroup, "WindowSwitcher");
>   defaultsConfigGroup.writeEntry("LayoutName", systemCG.readEntry("LayoutName", QStringLiteral("org.kde.breeze.desktop")));
>   
>   defaultsConfigGroup = KConfigGroup(&defaultsConfigGroup, "DesktopSwitcher");
>   defaultsConfigGroup.writeEntry("LayoutName", systemCG.readEntry("DesktopLayout", QStringLiteral("org.kde.breeze.desktop")))

gah, that's what i forgotten,
that the config key is different between kwinrc and the defaults file :/

meh, meh, meh

REPOSITORY
  rPLASMASDK Plasma SDK

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

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

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


More information about the Plasma-devel mailing list