D3805: Per-activity favorites (Final, again?)
Eike Hein
noreply at phabricator.kde.org
Mon Jul 3 08:37:02 UTC 2017
hein added inline comments.
INLINE COMMENTS
> main.xml:48
> + <entry name="favoritesPortedToKAstats" type="Bool">
> + <label>Are the favorites ported to use KActivitiesStats to allow per-activity favorites</label>
> + <default>false</default>
Pedantic: Broken grammar
> SideBarItem.qml:59
> onActionClicked: {
> + console.log(actionId + "<------------------ this is the id");
> actionTriggered(actionId, actionArgument);
Not a fan of debug stuff that's not really understandable without context ... lets take the console.log() calls out when finalizing the patch.
REVISION DETAIL
https://phabricator.kde.org/D3805
To: ivan, mart, hein
Cc: Zren, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170703/9cda06a5/attachment-0001.html>
More information about the Plasma-devel
mailing list