Review Request 111722: Do not select items unexpectedly when expanding a directory which has been copied and pasted together with its contents

Emmanuel Pescosta emmanuelpescosta099 at gmail.com
Sun Jul 28 13:04:13 BST 2013


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

Ship it!


Ship it after fixing the issue!

- Emmanuel Pescosta


On July 26, 2013, 3:44 p.m., Frank Reininghaus wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/111722/
> -----------------------------------------------------------
> 
> (Updated July 26, 2013, 3:44 p.m.)
> 
> 
> Review request for Dolphin.
> 
> 
> Description
> -------
> 
> The "Select all pasted items, even if they are not all created at the same time" feature caused a regression:
> 
> 1. mkdir test && cd test && mkdir a b && touch a/1 a/2 && dolphin .
> 2. Switch to Details View, expand "a", select a, a/1, a/2, and copy (or drag) them.
> 3. Open another folder and paste (or drop) the items.
> 4. Move around in the destination folder (a is not selected any more then).
> 5. Expand a -> a/1 and a/2 become selected.
> 
> IMHO, this is unexpected and should be fixed. We can do that with KDirModel::simplifiedUrlList(urls), which we also use in other places to remove children from the list if their parents are already in the list.
> 
> 
> Diffs
> -----
> 
>   dolphin/src/views/dolphinview.cpp d54a101 
> 
> Diff: http://git.reviewboard.kde.org/r/111722/diff/
> 
> 
> Testing
> -------
> 
> Works for me, and I haven't seen any regressions yet.
> 
> 
> Thanks,
> 
> Frank Reininghaus
> 
>

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


More information about the kfm-devel mailing list