D8493: Make Folder View screen aware

Andras Mantia noreply at phabricator.kde.org
Wed Nov 15 11:11:19 UTC 2017


amantia added inline comments.

INLINE COMMENTS

> mwolff wrote in screenmapper.cpp:41
> this is a pretty arbitrary timer, can you add a comment on why 100ms is better than going through the eventloop once via QMetaObject::invokeMethod with the delayed flag set?

Done. InvokeMethod would not help, as the idea here is to get only one screenMappingChanged instead of e.g 50 when FolderModel gets the entries on startup.

REPOSITORY
  R119 Plasma Desktop

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

To: amantia, #plasma, ervin, mlaurent, dvratil, hein, aacid, davidedmundson, apol, mwolff
Cc: anthonyfieroni, ngraham, mwolff, davidedmundson, broulik, mart, plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171115/8b939b6b/attachment.html>


More information about the Plasma-devel mailing list