D5504: Introduce a HasLauncher data role.

[David Edmundson]

davidedmundson added inline comments.

INLINE COMMENTS

> tasksmodel.cpp:213
> +            if (roles.contains(AbstractTasksModel::AppId) || roles.contains(AbstractTasksModel::LauncherUrlWithoutIcon)) {
> +                q->dataChanged(q->index(0, 0), q->index(q->rowCount() - 1, 0),
> +                    QVector<int>{AbstractTasksModel::HasLauncher});

Edit:

Can't your just connect to "this" for the dataChanged

and then you can re-use the topLeft and bottomRight indexes

REPOSITORY
  R120 Plasma Workspace

BRANCH
  master

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

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