Review Request 109900: Show the number of items of every group in the group headers

[Frank Reininghaus]

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


Thanks for the patch, but as I said earlier: large changes should really be discussed in advance or at least be explained properly. It's not clear at all to me why a new class is needed for this.

- Frank Reininghaus


On April 7, 2013, 9:21 p.m., Emmanuel Pescosta wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109900/
> -----------------------------------------------------------
> 
> (Updated April 7, 2013, 9:21 p.m.)
> 
> 
> Review request for Dolphin and Frank Reininghaus.
> 
> 
> Description
> -------
> 
> Show the number of items of every group in the group headers
> 
> See attached screenshot
> 
> 
> Diffs
> -----
> 
>   dolphin/src/CMakeLists.txt ffb232c 
>   dolphin/src/kitemviews/kfileitemlistgroupheader.h PRE-CREATION 
>   dolphin/src/kitemviews/kfileitemlistgroupheader.cpp PRE-CREATION 
>   dolphin/src/kitemviews/kfileitemlistview.h d795c96 
>   dolphin/src/kitemviews/kfileitemlistview.cpp b7f7455 
>   dolphin/src/kitemviews/kstandarditemlistgroupheader.h 26158d6 
>   dolphin/src/kitemviews/kstandarditemlistgroupheader.cpp 1e23c0a 
> 
> Diff: http://git.reviewboard.kde.org/r/109900/diff/
> 
> 
> Testing
> -------
> 
> Works. All tests passed.
> 
> Also correct item count after creating/deleting items.
> 
> 
> File Attachments
> ----------------
> 
> 
>   http://git.reviewboard.kde.org/media/uploaded/files/2013/04/07/snapshot1.png
> 
> 
> Thanks,
> 
> Emmanuel Pescosta
> 
>

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