After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 340321 - Nautilus should use Trash icon for Move to Trash menu items
Nautilus should use Trash icon for Move to Trash menu items
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: [obsolete] Visual Design
0.x.x [obsolete]
Other Linux
: Normal normal
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-05-01 18:40 UTC by Rodney Dawes
Modified: 2006-07-25 01:39 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to use user-trash-empty rather than GTK_STOCK_DELETE (1.84 KB, patch)
2006-05-01 18:40 UTC, Rodney Dawes
none Details | Review
Updated patch to fix more uses of delete instead of trash (4.19 KB, patch)
2006-07-10 19:21 UTC, Rodney Dawes
none Details | Review

Description Rodney Dawes 2006-05-01 18:40:03 UTC
Nautilus currently uses GTK_STOCK_DELETE for the "Move to Trash" menu items in its UI. These should be replaced with the trash icon from the icon theme, as the delete icon should contain a different metaphor that does not imply that items are recoverable after being deleted. The attached patch does this by switching from GTK_STOCK_DELETE to "user-trash-empty" which is in the Icon Naming Specification and provided by gnome-icon-theme 2.15.x.
Comment 1 Rodney Dawes 2006-05-01 18:40:52 UTC
Created attachment 64629 [details] [review]
Patch to use user-trash-empty rather than GTK_STOCK_DELETE
Comment 2 Sebastien Bacher 2006-05-26 22:37:22 UTC
as pointed on https://launchpad.net/distros/ubuntu/+source/tangerine-icon-theme/+bug/45925 the icon of the context menu of the location button of the spatial mode needs to be changed too
Comment 3 Rodney Dawes 2006-07-10 19:21:46 UTC
Created attachment 68738 [details] [review]
Updated patch to fix more uses of delete instead of trash
Comment 4 Rodney Dawes 2006-07-10 19:22:11 UTC
Is this better? Can we get it in soon? :)
Comment 5 Martin Wehner 2006-07-24 23:41:00 UTC
Thanks, it's on HEAD:

2006-07-25  Martin Wehner  <martin.wehner@gmail.com>

	* libnautilus-private/nautilus-icon-factory.c:
	* src/file-manager/fm-directory-view.c:
	* src/file-manager/fm-tree-view.c: (create_popup_menu):
	* src/nautilus-window-menus.c:
	Use trash icons from the icon naming specification for trash
	related operations. (#340321)

	Patch from Rodney Dawes  <dobey@novell.com>
Comment 6 Luca Ferretti 2006-07-25 00:02:14 UTC
Rodney, what about bug # 332448 ('Delete from Trash' menu item should not have the trashcan icon) ?
Comment 7 Rodney Dawes 2006-07-25 01:39:30 UTC
Luca, that is a dup of #169451 I think. That menu item should be using GTK_STOCK_DELETE, but the icon is currently a trash icon, in the theme. We need to make this a better metaphor there.