Review Request 119220: Figure out why my plasma wasn't notifying the startup

Aleix Pol Gonzalez aleixpol at kde.org
Mon Jul 14 11:24:08 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119220/
-----------------------------------------------------------

(Updated July 14, 2014, 11:24 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-framework


Description
-------

Make sure AppletPrivate::uiReady is set in applet_p.cpp when we report that the ui is ready.
Make sure that if we loop through all the containments and they're all ready, we emit that it's done.

So far, Corona::startupCompleted was never emitted.


Diffs
-----

  src/plasma/corona.h baa9ee9 
  src/plasma/corona.cpp e18f081 
  src/plasma/private/applet_p.cpp 4fe27c9 

Diff: https://git.reviewboard.kde.org/r/119220/diff/


Testing
-------


File Attachments
----------------

pf-startup.patch
  https://git.reviewboard.kde.org/media/uploaded/files/2014/07/11/e5b15661-6507-441e-94cc-ed7828ab0ff9__pf-startup.patch


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140714/2ddbe6d3/attachment.html>


More information about the Plasma-devel mailing list