D8598: FolderView: position files at drop event target position

[Eike Hein]

hein accepted this revision.
hein added a comment.


  Small comment change request in the comments, but code-wise good to go.

INLINE COMMENTS

> foldermodel.cpp:1068
> +     *
> +     * TODO: move this somehow to the Positioner
> +     */

As mentioned on IRC I'd currently prefer to keep the Positioner sort of optional architecturally. The idea is that non-containment FolderViews shouldn't need or use the Positoner (though this isn't the case currently in the code - it will always use it, just with an empty mapping) as they don't support sparse positioning and can scale better. Will also be relevant for future uses of this code like the Plasma Mobile file manager. Therefore I think this TODO note should be removed.

REPOSITORY
  R119 Plasma Desktop

REVISION DETAIL
  https://phabricator.kde.org/D8598

To: mwolff, hein, amantia
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171122/c0467b11/attachment.html>