Review Request 112312: Make thumbnail generation more responsive to scrolling/paging

Christoph Feck christoph at maxiom.de
Fri Aug 30 11:58:26 BST 2013


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

(Updated Aug. 30, 2013, 10:58 a.m.)


Status
------

This change has been discarded.


Review request for Dolphin.


Description
-------

The thumbnail generation is delayed by several timers to make the user interface more responsive. During scrolling, however, it delayed the generation by 300 ms, which I felt to long.

Looking at the code, this long interval is supposed to be used for zooming, where previously generated thumbnails are recreated at a new size, and moving the zoom slider would otherwise recreated them too often.

But the long interval is also used for scrolling, because the thumbnail generation is broken up into multiple chunks, and only the first chunk used the short interval. If you scroll down to the place where no thumbnails have been generated yet, it waits for a long time before resuming.

Suggested patch fixes this.


This addresses bug 322093.
    http://bugs.kde.org/show_bug.cgi?id=322093


Diffs
-----

  dolphin/src/kitemviews/kfileitemlistview.h 49ff773 
  dolphin/src/kitemviews/kfileitemlistview.cpp 1f0fcbd 

Diff: http://git.reviewboard.kde.org/r/112312/diff/


Testing
-------


Thanks,

Christoph Feck

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.kde.org/mailman/private/kfm-devel/attachments/20130830/67cb835f/attachment.htm>


More information about the kfm-devel mailing list