D5504: Introduce a HasLauncher data role.
noreply at phabricator.kde.org
Wed Apr 19 12:00:22 UTC 2017
davidedmundson added a comment.
appsMatch depends on AppId and LauncherURLWithoutIcon
You're handling launcherTasksModel changing, but can we rely on this data being static for a proxyIndex?
If not, we need to connect to TasksModel::dataChanged for those roles and potentially then emit that HasLauncher changed.
Having read your other patch you could argue that you don't need to signal this role as you (currently) only use this role from method calls triggered by other events....but if you take that POV, there's no point doing the launcherListChanged connect.
R120 Plasma Workspace
To: hein, #plasma, davidedmundson
Cc: plasma-devel, spstarr, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Plasma-devel