Bug 157848

Summary:	Trash icon does not function
Product:	[Unmaintained] plasma4	Reporter:	tkaitchuck
Component:	desktop	Assignee:	Plasma Bugs List <plasma-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal
Priority:	NOR
Version First Reported In:	unspecified
Target Milestone:	---
Platform:	unspecified
OS:	Linux
Latest Commit:		Version Fixed In:
Sentry Crash Report:

Description tkaitchuck 2008-02-15 05:01:50 UTC

Version:           1.0.1 (using 4.0.1 (KDE 4.0.1), Debian packages)
Compiler:          cc
OS:                Linux (x86_64) release 2.6.22-3-amd64

If I have a trash icon on my desktop, that is leftover form KDE3. Or if I try to get one by draging one from my Places or the K-Menu, I have a trash icon on the desktop, but it does not exhibit the expected behavior of being able to drag icons there and have them go into the trash. Neither can I right click on it to empty the trash. The only thing this allows me to do is to look into the trash.

Comment 1 Chani 2008-02-15 10:47:40 UTC

plasma has its own trash applet - it was in playground, but I think it's in kdereview right now. someone's planning to move it into kdebase soon.
you should delete that icon and use the trash applet.

Comment 2 Aaron J. Seigo 2008-02-15 10:56:51 UTC

Chani: indeed; however, the icon applet probably needs much better drop support in general. it should open files with the associated application if it's an app; offer to copy/move the file to the folder if its a folder URL, etc...

Comment 3 Aaron J. Seigo 2008-02-15 11:02:01 UTC

hm.. looking at the code it already does this, actually. it's just trash:/ and other such protocols that don't work. looking into it now ....

Comment 4 Aaron J. Seigo 2008-02-15 11:21:04 UTC

SVN commit 775266 by aseigo:

* enable proper dropping on desktop files that contain paths to directories (also works for, e.g. trash:/)
* plug several memory leaks

ruphy: this is a good candidate for backporting. no strings, numerous bug fixes.

CCMAIL:riccardo@kde.org
BUG:157848


 M  +27 -9     icon.cpp   [POSSIBLY UNSAFE: KRun::runCommand]


WebSVN link: http://websvn.kde.org/?view=rev&revision=775266

Comment 5 Marco Martin 2008-02-15 12:26:46 UTC

maybe when the applet will be in base add it instead of an icon applet when a trash:/ url is dragged in the desktop? (i know, special cases are evil, but otherwise reports like this will keep coming because it's not so obvious)