Review Request 115709: Monitor the QQmlParser status from dialog

David Edmundson david at davidedmundson.co.uk
Thu Feb 13 10:27:12 UTC 2014


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

(Updated Feb. 13, 2014, 10:27 a.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Repository: plasma-framework


Description
-------

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.


Diffs
-----

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

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


Testing
-------


Thanks,

David Edmundson

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


More information about the Plasma-devel mailing list