D8723: When used by QML, only populate after component is complete

[David Edmundson]

davidedmundson added a comment.


  > I'm not super happy with this behavior change for C++ users,
  
  I've seen other people export a subclass for QtQuick  that sets the usedByQml flag.
  
  I love QQmlParserStatus so +1

INLINE COMMENTS

> tasksmodel.cpp:632
>  {
> +    qDebug() << usedByQml << componentComplete;
> +

.

> tasksmodel.cpp:923
> +    // directly.
> +    QTimer::singleShot(0, [this]() {
> +        d->updateGroupInline();

may as well QTimer:;singleShot(0, this, [this](...

just in case you get deleted in the same space.

REPOSITORY
  R120 Plasma Workspace

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

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