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 516007 - Use standard::description file attribute as window title if availible
Use standard::description file attribute as window title if availible
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: [obsolete] GIO
0.x.x [obsolete]
Other All
: Normal normal
: ---
Assigned To: Paweł Paprota
Nautilus Maintainers
Depends on:
Blocks:
 
 
Reported: 2008-02-12 13:10 UTC by Alexander Larsson
Modified: 2008-04-27 01:06 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
516007-use-standard_description-gio-attribute-when-available.patch (1.03 KB, patch)
2008-04-25 20:17 UTC, Paweł Paprota
rejected Details | Review
uses nautilus-file machinery more. (3.70 KB, patch)
2008-04-25 21:50 UTC, A. Walton
committed Details | Review

Description Alexander Larsson 2008-02-12 13:10:38 UTC
standard::description was recently added to gio for uses such as window title. It has the name plus optional extra info like "foo on $host" or "bar (in trash)".
Comment 1 Paweł Paprota 2008-04-25 20:17:07 UTC
Created attachment 109930 [details] [review]
516007-use-standard_description-gio-attribute-when-available.patch
Comment 2 A. Walton 2008-04-25 21:50:29 UTC
Created attachment 109937 [details] [review]
uses nautilus-file machinery more.

Here's the patch I futilely attempted to send to the list.
Comment 3 Christian Neumair 2008-04-25 22:21:22 UTC
Paweł, A.: Thanks for working on this!

Marking attachment 109930 [details] [review] by Paweł as "rejected", and patch 109937 by A. as "reviewed".

Paweł: A. is right, synchronous I/O should not be used.

A.: The patch looks fine.

I wonder why you changed NAUTILUS_FILE_DEFAULT_ATTRIBUTES, and appended ",mountable::*", though. It is already in the list, and is not related to the file description, since the description is in the "standard" namespace.
Comment 4 A. Walton 2008-04-25 23:57:25 UTC
I didn't add it, I removed it; mountable::* was there twice, I reduced it to one:

-	"standard::*,access::*,mountable::*,time::*,unix::*,owner::*,selinux::*,thumbnail::*,mountable::*"
+	"standard::*,access::*,mountable::*,time::*,unix::*,owner::*,selinux::*,thumbnail::*"

Seemed redundant to me ;) Good to commit?
Comment 5 Christian Neumair 2008-04-26 23:39:21 UTC
Yes. Sorry for the noise.
Comment 6 A. Walton 2008-04-27 01:06:39 UTC
2008-04-26  A. Walton  <awalton@gnome.org>

	reviewed by: Christian Neumair

	* libnautilus-private/nautilus-file-private.h:
	* libnautilus-private/nautilus-file-utilities.c
	(nautilus_compute_title_for_location):
	* libnautilus-private/nautilus-file.c (update_info_internal),
	(nautilus_file_get_description):
	* libnautilus-private/nautilus-file.h:
	Use standard::description in the window title when it's available.
	Closes bug #516007.

Committed, closing.