----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://git.reviewboard.kde.org/r/115709/#review49707 -----------------------------------------------------------
This review has been submitted with commit 7c135314aa9de3e474f1e482d61a415da29fffad by David Edmundson to branch master. - Commit Hook On Feb. 12, 2014, 10:43 p.m., David Edmundson wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://git.reviewboard.kde.org/r/115709/ > ----------------------------------------------------------- > > (Updated Feb. 12, 2014, 10:43 p.m.) > > > 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 > >
_______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel