Bug 157848 - Trash icon does not function
Summary: Trash icon does not function
Status: RESOLVED FIXED
Alias: None
Product: plasma4
Classification: Plasma
Component: desktop (show other bugs)
Version: unspecified
Platform: unspecified Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-02-15 05:01 UTC by tkaitchuck
Modified: 2008-02-15 12:26 UTC (History)
0 users

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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)