D21513: queue load() after all deleteLater() in unload() are done

Marco Martin noreply at phabricator.kde.org
Fri May 31 17:38:14 BST 2019


mart created this revision.
mart added a reviewer: Plasma.
Herald added a project: Plasma.
Herald added a subscriber: plasma-devel.
mart requested review of this revision.

REVISION SUMMARY
  Put load in queue of the event loop to wait for the whole set of containments to have been deleteLater(), as some like FolderView operate on singletons which can cause inconsistent states

TEST PLAN
  now switching look and feel between 2 themes that use both FolderView
  as desktop containment, the icons are still there

REPOSITORY
  R120 Plasma Workspace

BRANCH
  phab/queueload

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

AFFECTED FILES
  shell/shellcorona.cpp

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


More information about the Plasma-devel mailing list