graesslin added a comment.
can you explain why it should be configurable?
INLINE COMMENTS
> osd.cpp:36
> , m_osdPath(corona->lookAndFeelPackage().filePath("osdmainscript"))
> + ,
> m_osdConfig(KConfigGroup(KSharedConfig::openConfig(QStringLiteral("plasmarc")),
> QStringLiteral("OSD")))
> {
sharing a lesson learned from KWin: if you dependency inject the KSharedConfig
you get two advantages:
1. No stringly-typed names like plasmarc
2. the class is easier to test as in a unit test you can inject a dummy
KSharedConfig exactly with the key/values as one needs.
REPOSITORY
rPLASMAWORKSPACE Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D1770
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: broulik, #plasma
Cc: graesslin, plasma-devel, sebas
_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel
