D12936: kcm_workspace is finished.

Nathaniel Graham noreply at phabricator.kde.org
Thu May 17 00:51:08 UTC 2018


ngraham added inline comments.

INLINE COMMENTS

> kcm.cpp:133
> +// If yes, setNeedsSave(true).
> +void KCMWorkspaceOptions::handleIfNeedSave()
> +{

How about `handleNeedsSave()` instead?

> furkantokac wrote in ToolTip.qml:2
> Yeap it's Roman's implementation used in "input" kcm. Shouldn't I use it ?

If this code is used in multiple KCMs, we shouldn't duplicate it; we should upstream it so that it only needs to exist in one place at a time.

REPOSITORY
  R119 Plasma Desktop

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

To: furkantokac, romangg, ngraham
Cc: davidedmundson, plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180517/882d4b07/attachment.html>


More information about the Plasma-devel mailing list