Review Request: Restore ability to select a groups of files in Dolphin by clicking a group title

Todd toddrme2178 at gmail.com
Sun Jan 31 17:06:55 GMT 2010



> On 2010-01-31 10:15:17, Peter Penz wrote:
> > Thanks for the update, please commit!
> > 
> > BTW: the "@since 4.5" in private methods is not necessary, only for public + protected methods.
> 
> Todd wrote:
>     I don't have SVN access.  Can someone else commit it?  Also, should this be backported to 4.4?  I know it is a pretty large and intrusive patch, but on the other hand it also fixes what, in my opinion, is a pretty major feature regression.
> 
> Peter Penz wrote:
>     I've tested your patch and committed it. I've also backported it to 4.4, as it is a regression in comparison to 4.3 if this patch is not supplied. Thanks again for your contribution, this was a very important fix!

Great!  Thank you very much.


- Todd


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


On 2010-01-30 23:19:23, Todd wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2764/
> -----------------------------------------------------------
> 
> (Updated 2010-01-30 23:19:23)
> 
> 
> Review request for Dolphin and kdelibs.
> 
> 
> Summary
> -------
> 
> This patch restores the ability to select a group of files in Dolphin by clicking on the group's title.  This is a feature regression between 4.3 and 4.4  Unlike the original version, which could only select groups, this implementation allows for selecting and deselecting groups (Peter Penz said he considered the lack of deselecting to be a bug).  
> 
> The patch also removes a workaround in Dolphin that does not appear to be necessary anymore and was preventing the group selection from working.  
> 
> I know this is probably not the prettiest solution, but it does work.  I made a couple of speed improvements in corner cases, which should be clear from the diff, but these could be removed without altering the functionality of the code.  I am open to suggestions on improvements.  I would also appreciate it if other people tested it as well.
> 
> 
> This addresses bug 214859.
>     https://bugs.kde.org/show_bug.cgi?id=214859
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdebase/apps/dolphin/src/dolphiniconsview.cpp 1082212 
>   /trunk/KDE/kdelibs/kdeui/itemviews/kcategorizedview.cpp 1082212 
>   /trunk/KDE/kdelibs/kdeui/itemviews/kcategorizedview_p.h 1082212 
> 
> Diff: http://reviewboard.kde.org/r/2764/diff
> 
> 
> Testing
> -------
> 
> Selected and deselected groups in Dolphin under various circumstances.
> 
> 
> Thanks,
> 
> Todd
> 
>





More information about the kde-core-devel mailing list