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