Improve multi-desktop folderview behavior

Authored by amantia on Mar 8 2018, 10:57 AM.

Description

Improve multi-desktop folderview behavior

Summary:

  1. remove the items from the active map when they are on a disabled screen.

Fixes the case of Screen 1 showing Foo, Screen 2 showing Bar, Screen 1
switching to Bar (files from under Foo should disappear), Screen 2 switches to
Foo (files from under Foo should appear now on screen 2, they did not before)

  1. don't store twice an item in the m_itemsOnDisablesScreensMap. This could

happen before with setups having more than two screens.

  1. Remove m_firstScreenForPath and instead store the screen ids per url in

m_screensPerPath and use the one with lowest id from it for the first screen.
Help with detecting the first valid screen in 2+ screen scenarios.

  1. remove the isEmpty check in addScreen, that caused all items to be added

under a screen with "desktop:///" path, even if they belonged somewhere else.

  1. Save and restore the items-on-disabled-screen map to the config file.

This should help in having a consistent behavior after restarting plasma.
It could help with bug 389745

BUG: 390676
CCBUG: 389745

Reviewers: Plasma, hein

Reviewed By: Plasma, hein

Subscribers: ngraham, plasma-devel

Tags: Plasma

Differential Revision: https://phabricator.kde.org/D10728

Details

Committed
amantiaMar 8 2018, 10:57 AM
Reviewer
Plasma
Differential Revision
D10728: Improve multi-desktop folderview behavior
Parents
R119:cb1a2f8e70a0: Use QUrl in the ScreenMapper API
Branches
Unknown
Tags
Unknown