D14243: Fix bad model hygiene in Positioner::move()

[Eike Hein]

hein added a comment.


  I probably should at some point. Unfortunately it also got a lot more complicated with the addition of multi-screen support.
  
  But basically, it's a proxy in use when Folder View is used on the desktop and allows moving icons around. The view is a Grid View. The source model is a list. It maps from positions the user drops things to to rows in the source model. The proxy maps only concern themselves with positions that map to real entries in the source model. The sparse gaps inbetween icons are not part of the maps, they just return empty data and `true` for the special "IsBlank" data role. The delegate implementation in the view is behind a Loader that does nothing when IsBlank is true.
  
  Multiscreen works by further associating source rows with a screen and filtering in a screen-dependent way.
  
  There's some other things it needs to do, such as when a new file appears it needs to figure out the first free space and map it there, etc.

REPOSITORY
  R119 Plasma Desktop

BRANCH
  master

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

To: hein, davidedmundson
Cc: plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20180801/cd376f90/attachment.html>