Bug 398141

Summary:	Dropping an Icon from Screen1 onto Screen2 opens the copy/cut contextmenu when Screen2 uses "desktop:/"
Product:	[Plasma] plasmashell	Reporter:	Chris Holland <zrenfire>
Component:	Desktop icons & Folder View widget	Assignee:	Eike Hein <hein>
Status:	RESOLVED FIXED
Severity:	normal	CC:	jeffclay, kde, nate, plasma-bugs-null, rdieter
Priority:	NOR
Version First Reported In:	5.13.4
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description Chris Holland 2018-09-02 02:39:15 UTC

A fellow on IRC reported this bug. He was Using Fedora 28.

I only have 1 screen, so I tested 2 screens with a KDE Neon iso in Virtualbox.

I tried changing the first checkbox from "show the desktop folder" to "Select a Place > Desktop" and the contextmenu did not show up.

https://streamable.com/ufxx4

So it seems to be a bug with droping from/to the url "desktop:/", where as dropping onto the "/home/zren/Desktop" url is fine.

I tested the other two entries in the "logic table" too.

https://streamable.com/9nfox

* Dragging from "/home/zren/Desktop" to "desktop:/" caused a contextmenu to appear.
* Dragging from "desktop:/" to  "/home/zren/Desktop" caused nothing to happen. First of all, the two urls are detected as "different folders" so the same icons appear on both screens. The icon on the second screen does not move, nor does the icon on the first screen. No contextmenu appears either.

The IRC user confirmed the workaround of using "Places > Desktop" on all 3 of his screens.
Note that this isn't an ideal workaround as *.desktop files appear as "files" instead of "links".

Comment 1 Kai Uwe Broulik 2018-09-07 13:13:44 UTC

So we should filter out Places > Desktop from the dropdown then.

Comment 2 Kai Uwe Broulik 2018-09-10 10:59:31 UTC

Git commit 45c420b7fd1115f569ffdcdb1be308e1ecf8cdc6 by Kai Uwe Broulik.
Committed on 10/09/2018 at 10:58.
Pushed by broulik into branch 'master'.

[Folder View] Hide /home/foo/Desktop place in configuration

Folder View is meant to use desktop:/ and having two entries that behave differently can generate confusion.

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

M  +1    -0    containments/desktop/package/contents/ui/ConfigLocation.qml
M  +26   -0    containments/desktop/plugins/folder/placesmodel.cpp
M  +6    -0    containments/desktop/plugins/folder/placesmodel.h

https://commits.kde.org/plasma-desktop/45c420b7fd1115f569ffdcdb1be308e1ecf8cdc6