Review Request 109935: Make KFilePlacesModel treats internal moves by drag'n'drop as moves instead of remove+insert

[Kevin Ottens]

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

Ship it!


Ship It!

- Kevin Ottens


On April 9, 2013, 3:56 p.m., Aurélien Gâteau wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109935/
> -----------------------------------------------------------
> 
> (Updated April 9, 2013, 3:56 p.m.)
> 
> 
> Review request for kdelibs and Kevin Ottens.
> 
> 
> Description
> -------
> 
> When user moves an item from the model by dragging it, KFilePlacesModel removes the item row and adds it back at the new position. This patch changes this behavior to use beginMoveRows() and endMoveRows() instead.
> 
> I came upon this while implementing drag'n'drop to reorder items in Homerun. Homerun uses KFilePlacesModel in the "favorite places" source, and the reordering code expects the model to use beginMoveRows() and endMoveRows().
> 
> 
> Diffs
> -----
> 
>   kfile/kfileplacesmodel.cpp 0192926233bac62596e4c04b9267229f4a284c95 
>   kfile/tests/kfileplacesmodeltest.cpp 2ae9b7548e47985369dc896086031c07bf0c5a4c 
> 
> Diff: http://git.reviewboard.kde.org/r/109935/diff/
> 
> 
> Testing
> -------
> 
> Tested with Homerun and other applications using KFileDialog. Updated the unit-tests.
> 
> 
> Thanks,
> 
> Aurélien Gâteau
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20130410/eccdff9a/attachment.htm>