amantia created this revision.
amantia added reviewers: Plasma, ervin, mlaurent, dvratil, hein, aacid.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.

REVISION SUMMARY
  When using multiple screen with a Folder View as a desktop containment, we 
need to make sure a file belongs only to one screen. The patch adds support for 
this by:
  
  1. Introducing a ScreenMapper object shared acrossed all Folder Views
  2. FolderModel registers the screen it resides on if used as a containment
  3. FolderModel filters out items not on the current screen (if used as a 
containment)
  4. FolderModel adds mapping for the newly appeared files. The new files will 
go to the registered screen having the smallest id. This also means by default 
all files appear on the screen with the smallest id.
  5. url/screen mapping is stored in the configuration of each folder view 
applet. This means duplication of the information, and although they should be 
in sync unless manually modified, the logic is that the last view's mapping 
option is used.

REPOSITORY
  R119 Plasma Desktop

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

AFFECTED FILES
  containments/desktop/package/contents/config/main.xml
  containments/desktop/package/contents/ui/FolderView.qml
  containments/desktop/package/contents/ui/FolderViewLayer.qml
  containments/desktop/plugins/folder/CMakeLists.txt
  containments/desktop/plugins/folder/foldermodel.cpp
  containments/desktop/plugins/folder/foldermodel.h
  containments/desktop/plugins/folder/folderplugin.cpp
  containments/desktop/plugins/folder/folderplugin.h
  containments/desktop/plugins/folder/screenmapper.cpp
  containments/desktop/plugins/folder/screenmapper.h

To: amantia, #plasma, ervin, mlaurent, dvratil, hein, aacid
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, 
abetts, sebas, apol, mart

Reply via email to