Review Request 110672: Allow multiple selection in archiveview

[Alim Gokkaya]

> On Aug. 16, 2014, 5:59 p.m., Albert Astals Cid wrote:
> > You know what would be awesome? An autotest, do you know how to use qtest? Do you think you could craft an autotest to test this? This way even it'd be easier to approve this. If not, i can try crafting one myself, since honestly after the previous fiasco i don't feel confortable approving this "relatively big" change in a code i don't know much.

Ok, let's hear what Raphael thinks and I'll try and write a test case in the meantime. I don't know anything about qtest yet but i've tested this manually with a few different scenarios like the one Raphael told.


- Alim


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/110672/#review64644
-----------------------------------------------------------


On Aug. 16, 2014, 11:13 a.m., Alim Gokkaya wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/110672/
> -----------------------------------------------------------
> 
> (Updated Aug. 16, 2014, 11:13 a.m.)
> 
> 
> Review request for KDE Utils, Albert Astals Cid and Raphael Kubo da Costa.
> 
> 
> Repository: ark
> 
> 
> Description
> -------
> 
> This patch enables extracting multiple files using drag&drop by removing the code that singles the selection.
> 
> 
> Diffs
> -----
> 
>   part/archiveview.cpp 6b9918d 
>   part/part.cpp b4ebcd2 
> 
> Diff: https://git.reviewboard.kde.org/r/110672/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Alim Gokkaya
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-utils-devel/attachments/20140816/376a657c/attachment.html>