Review Request 111956: Fix the problem that scrolling is extremely slow when hidden files are shown

[Commit Hook]

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


This review has been submitted with commit 381b17964d49259955eb5c5e537d290b26c162ef by Frank Reininghaus to branch KDE/4.11.

- Commit Hook


On Aug. 8, 2013, 9:31 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111956/
> -----------------------------------------------------------
> 
> (Updated Aug. 8, 2013, 9:31 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Description
> -------
> 
> The problem was that
> 
> KIconLoader::global()->drawOverlays(overlays, m_pixmap, KIconLoader::Desktop);
> 
> which is quite slow, is called every time just before an item with overlays appears on the screen. We can fix that by extending the cool icon caching that Emmanuel implemented some time ago: cache not only the pixmaps for the icons, but the pixmaps for icon+overlay. This fixes the problem for the icons.
> 
> For the previews, I think it's best to apply the overlays in KFileItemModelRolesUpdater. This ensures that the overlays are applied just once, and not every time the preview is shown on the screen.
> 
> The "apply overlays" code is duplicated now, but I still think that this is the cleanest solution. If anyone has better ideas, please let me know!
> 
> 
> This addresses bugs 310662 and 314339.
>     http://bugs.kde.org/show_bug.cgi?id=310662
>     http://bugs.kde.org/show_bug.cgi?id=314339
> 
> 
> Diffs
> -----
> 
>   dolphin/src/kitemviews/kfileitemmodelrolesupdater.cpp 698a6c5 
>   dolphin/src/kitemviews/kstandarditemlistwidget.h ecf3a3b 
>   dolphin/src/kitemviews/kstandarditemlistwidget.cpp b429211 
> 
> Diff: http://git.reviewboard.kde.org/r/111956/diff/
> 
> 
> Testing
> -------
> 
> Scrolling is much faster when hidden items and symlinks are shown. The overlays are still applied correctly for icons and previews.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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