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 313172 - Multiple glitches in Trash
Multiple glitches in Trash
Status: RESOLVED OBSOLETE
Product: nautilus
Classification: Core
Component: Trash
2.16.x
Other All
: Normal minor
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2005-08-11 00:11 UTC by Marc E.
Modified: 2009-02-21 14:57 UTC
See Also:
GNOME target: ---
GNOME version: 2.15/2.16


Attachments
Number one (73.43 KB, image/png)
2005-08-11 00:11 UTC, Marc E.
  Details
Number three, nautilus (29.84 KB, image/png)
2005-08-11 00:12 UTC, Marc E.
  Details
Extra missing icon (13.57 KB, image/png)
2005-08-11 00:12 UTC, Marc E.
  Details
This patch adds the icon to the "Empty Trash Button". (739 bytes, patch)
2006-02-12 05:00 UTC, Lakin Wecker
none Details | Review
A patch to remove the "Volume:" and "Free space:" items from the properties page. (615 bytes, patch)
2006-02-12 05:28 UTC, Lakin Wecker
none Details | Review

Description Marc E. 2005-08-11 00:11:08 UTC
Please describe the problem:
There are many "cosmetic" problems in the Trash.

The first one is being the properties window, is it really needed to have a
Properties page?, ok, it might be justified, but a Notes page?. (See attachment
number one)

Secondly, is that the properties window doesnt even work right (See attachment
number two)

Third, is that the nautilus File menu doesnt display the Property page menuitem
as sensitive. (See attachment number three)

I guess this opens a discussion about the points described above. Wasn't it
going to be replaced by the trash applet?

Steps to reproduce:


Actual results:


Expected results:


Does this happen every time?


Other information:
Comment 1 Marc E. 2005-08-11 00:11:46 UTC
Created attachment 50543 [details]
Number one
Comment 2 Marc E. 2005-08-11 00:12:30 UTC
Created attachment 50544 [details]
Number three, nautilus
Comment 3 Marc E. 2005-08-11 00:12:47 UTC
Created attachment 50545 [details]
Extra missing icon
Comment 4 Marc E. 2005-08-11 00:13:45 UTC
Sorry, in the report, attachment one and two, are the same, (See attachment
number one).

The second attachment should be (Extra missing icon), and it is about the
missing icon and text in the empty trash dialog.
Comment 5 Christian Neumair 2005-08-12 22:15:53 UTC
Thanks for your bug report!

> is it really needed to have a Properties page

You're raising a good question. I think it is considered useful to see the
overall amount of space used by items in trash.

> but a Notes page?

Does it do any harm?

> Third, is that the nautilus File menu doesnt display the Property page menuitem
as sensitive.

Well, the Nautilus maintainers think that the actions in the "File" menu should
just refer to the container (i.e. the trash) contents, not to the container
itself. This means consequently that the "Properties" item is only meant to be
used for trash contents. In Nautilus 2.11, you can however use the context menu
on the "Location:" label (navigational) or the location button (spatial) to
bring up a menu with container-related actions.

I agree with you that the "Empty Trash" dialog button sould contain an
GTK_STOCK_CLEAR pixbuf.
Comment 6 Lakin Wecker 2006-02-12 05:00:47 UTC
Created attachment 59166 [details] [review]
This patch adds the icon to the "Empty Trash Button".

This patch adds the stock GTK_STOCK_CLEAR to the Empty Trash button on the dialog which confirms the emptying of the trash. This at least partially fixes the problems described.
Comment 7 Lakin Wecker 2006-02-12 05:28:06 UTC
Created attachment 59168 [details] [review]
A patch to remove the "Volume:" and "Free space:" items from the properties page.

A patch to remove the "Volume:" and "Free space:" items from the properties page.  I agree with the reporter, this are not necessary.  The trash is a merged directory which shows items which are possibly located on multiple volumes which each have their own free-space values, so showing them in the Trash directory properties page is un-necessary, IMHO.
Comment 8 Marc E. 2006-02-12 07:09:54 UTC
It took 6 months to do a simple thing, but it was done.

Good job. I believe this wont even make it for 2.14 am I right?
Comment 9 Marc E. 2006-02-12 07:20:37 UTC
Also I just realized that on .12, the MenuItem for "Empty Trash" also needs a STOCK_CLEAR, and the "Move to Trash, and Delete" MenuItems should outright be taken out, not set unsensitive.

I agree with Lakin on the properties page, it is unnecesary.
Comment 10 Marc E. 2006-05-06 21:31:10 UTC
Please update to HEAD.
Comment 11 Christian Neumair 2006-05-06 21:50:26 UTC
Updated.
Comment 12 A. Walton 2008-05-23 18:34:33 UTC
The commit of Revision 14188 [1] obsoletes the need for the latter patch, but the former one might still be valid (though it doesn't apply anymore, apparently for a long time). 

Setting status on these.

[1]: http://svn.gnome.org/viewvc/nautilus?view=revision&revision=14188
Comment 13 André Klapper 2008-07-16 17:00:43 UTC
Is this still an issue in 2.22? Trash handling has been rewritten.
Comment 14 Tobias Mueller 2009-02-21 14:57:43 UTC
I assume it's not. Please reopen if this is still an issue!