[Differential] [Commented On] D4749: Fix another regression with rearranging launchers in an activities world.

Kai Uwe Broulik noreply at phabricator.kde.org
Fri Feb 24 09:44:31 UTC 2017


broulik added a comment.


  With the typo fixed, I can now re-arrange launchers and it sticks when I launch the app.

INLINE COMMENTS

> launchertasksmodel.cpp:453
> +    if (newLaunchersOrder != d->launchersOrder
> +        || newActivitiesForLauncher != d->activitiesForLauncher)) {
>          // Common case optimization: If the list changed but its size

Superfluous parenthesis, doesn't compile.

REPOSITORY
  R120 Plasma Workspace

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

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

To: hein, #plasma, davidedmundson, mart
Cc: broulik, fvogt, plasma-devel, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170224/a5eef7fc/attachment.html>


More information about the Plasma-devel mailing list