Review Request 126163: [Widget Explorer] Make it feel a lot quicker

Kai Uwe Broulik kde at privat.broulik.de
Wed Nov 25 13:27:05 UTC 2015


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

(Updated Nov. 25, 2015, 1:27 p.m.)


Status
------

This change has been marked as submitted.


Review request for Plasma.


Changes
-------

Submitted with commit 7eb5de9feaf43d1b76c3f2d4e0b8ad9643135cf0 by Kai Uwe Broulik to branch Plasma/5.5.


Repository: plasma-desktop


Description
-------

Widget Explorer takes quite a while to open. Especially now that we load basically double the number of items (two columns) and significantly more complex SVGs. This patch uses a couple of tricks to make it feel a lot faster:
- Create context menu items on demand
- Set the model for the list after a slight delay
- Some minor cleanups

By setting the model later we let the dialog slide in first (KWin does the animation anyway so it won't block) and then nicely fade in the view once we have loaded it.

I have noticed that the applet item model which gets the decoration role from the applet package can take quite a while blocking the explorer. It might also be worth investigating whether we should keep the WidgetExplorer around when it has been opened once (I mean the model, not the QML view) so we save creating the model everytime we open it again.


Diffs
-----

  desktoppackage/contents/explorer/WidgetExplorer.qml 911d9c9 

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


Testing
-------

Feels a lot quicker now.


Thanks,

Kai Uwe Broulik

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


More information about the Plasma-devel mailing list