Review Request 115709: Monitor the QQmlParser status from dialog

David Edmundson david at
Wed Feb 12 22:43:47 UTC 2014

This is an automatically generated e-mail. To reply, visit:

Review request for Plasma.

Repository: plasma-framework


Monitor the QQmlParser status from dialog

This way we only call the expensive syncToMainItemSize once we have all
the information to accurately position the window.

The previous approach using the timer to limit the calls doesn't work entirely (according to my
qDebug inside the function) especially with the multiple threads to compile things.

This patch seems to prevent the dialog ever jumping around on load which I sometimes saw.


  src/declarativeimports/core/dialog.h 3c0df8b 
  src/declarativeimports/core/dialog.cpp b353caf 




David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <>

More information about the Plasma-devel mailing list