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