D20237: Port to new KWorkspace API

[Eike Hein]

hein added inline comments.

INLINE COMMENTS

> apol wrote in ItemGridDelegate.qml:33
> so model.enabled you want it to resolve to true? This will be hard to read to anyone who isn't you.
> 
> How about `model.hasOwnProperty("enabled") && model.enabled`? Or `typeof model.enabled === "undefined" || model.enabled`

For the record (so someone doesn't copy this idea), model.hasOwnProperty() wouldn't work to catch unimplemented roles in a model.

I'll do another approach you might like better.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

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

To: hein, #plasma, davidedmundson
Cc: broulik, apol, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190404/b25f0288/attachment.html>