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 586488 - attach distributor-logo as the filesystem icon
attach distributor-logo as the filesystem icon
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
2.27.x
Other All
: Normal enhancement
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 561926 (view as bug list)
Depends on: 620239
Blocks:
 
 
Reported: 2009-06-20 19:42 UTC by Nicolò Chieffo
Modified: 2011-08-22 13:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
use-distributor-logo.diff (1.11 KB, patch)
2009-06-20 19:47 UTC, Nicolò Chieffo
needs-work Details | Review
GVFS-use-distributor-logo.diff (796 bytes, patch)
2009-06-20 20:17 UTC, Nicolò Chieffo
none Details | Review
use (1.69 KB, patch)
2009-06-20 20:23 UTC, Nicolò Chieffo
none Details | Review
NAUTILUS-use-distributor-logo.diff (1.69 KB, patch)
2009-06-20 20:24 UTC, Nicolò Chieffo
none Details | Review
GVFS-use-distributor-logo.diff (739 bytes, patch)
2009-06-21 22:09 UTC, Nicolò Chieffo
none Details | Review
GVFS_drive_hardisk_root_icon.patch (521 bytes, patch)
2009-09-13 10:06 UTC, Nicolò Chieffo
rejected Details | Review
NAUTILUS_drive_hardisk_root_icon.patch (1.73 KB, patch)
2009-09-13 10:24 UTC, Nicolò Chieffo
none Details | Review
NAUTILUS_drive_hardisk_root_icon.patch (2.17 KB, patch)
2009-09-13 10:47 UTC, Nicolò Chieffo
needs-work Details | Review

Description Nicolò Chieffo 2009-06-20 19:42:30 UTC
I've created a small and simple patch, which attaches the distributor-logo icon to file:///
Comment 1 Nicolò Chieffo 2009-06-20 19:47:18 UTC
Created attachment 137081 [details] [review]
use-distributor-logo.diff

Due to the fact I didn't find it, the icon is missing in the computer:/// but it's present in places dialog. Please help
Comment 2 A. Walton 2009-06-20 19:51:25 UTC
The icon isn't used in computer: because computer: is a GVFS backend (gvfsd-computer). Nautilus really expects this to come from there and not shoehorned in here.

Furthermore, distributor-logo afaik isn't a widely accepted icon name, so it's likely that icon will be missing, resulting in the ugly not-found icon for everyone, so we'd need to specify fallbacks.
Comment 3 Nicolò Chieffo 2009-06-20 19:55:42 UTC
Thanks for the immediate answer. We can use themed icons with fallbacks, it's simple.
can you tell me if patching only gvfs would resolve also the places icon, or will we need also this patch?
Comment 4 Nicolò Chieffo 2009-06-20 20:17:35 UTC
Created attachment 137085 [details] [review]
GVFS-use-distributor-logo.diff
Comment 5 Nicolò Chieffo 2009-06-20 20:23:18 UTC
Created attachment 137086 [details] [review]
use
Comment 6 Nicolò Chieffo 2009-06-20 20:24:11 UTC
Created attachment 137087 [details] [review]
NAUTILUS-use-distributor-logo.diff

sorry, wrong name
Comment 7 Nicolò Chieffo 2009-06-20 20:26:37 UTC
it fallbacks first to "start-here" which is from the icon naming spec, and then to the original folder "driver-harddisk"
Comment 8 Nicolò Chieffo 2009-06-21 16:50:06 UTC
A. Walton, my work is finished. Do you accept those patches or should I make other changes?
Comment 9 André Klapper 2009-06-21 21:42:30 UTC
-  file->display_name = g_strdup (_("Filesystem"));
+  file->display_name = g_strdup (_("File System"));

Please revert this. Also see bug 586573.
Comment 10 Nicolò Chieffo 2009-06-21 22:07:18 UTC
Woops sorry! this was a previous change I forgot :(
Comment 11 Nicolò Chieffo 2009-06-21 22:09:08 UTC
Created attachment 137141 [details] [review]
GVFS-use-distributor-logo.diff

Ready
Comment 12 Marcus Carlson 2009-09-12 22:27:35 UTC
Ping, no chance for 2.28?
Comment 13 Vish 2009-09-13 06:12:35 UTC
The current patch uses only icons available in the present list of icons.
The drive will be displayed as a logo , which will not make it clear that the filesystem is a drive.
If the patch is applied , this would be a workaround and not a proper patch.

The ideal way would be to create a new icon name for the "Filesystem"
Either : drive-harddisk-root or drive-harddisk-dist

If it is absolutely essential to get this done in Ubuntu Karmic, it is easier to edit the gnome-dev-harddisk icon to have a logo than applying the current patch , which would affect all the distributions.
Comment 14 Nicolò Chieffo 2009-09-13 10:03:58 UTC
I'm rewriting the patches now, but we need the attention of the developers...

Another question to the developers is: why do I have to patch both nautilus and gvfs? don't you think that nautilus should use gvfs to get the filesystem icon?
Comment 15 Nicolò Chieffo 2009-09-13 10:06:50 UTC
Created attachment 143095 [details] [review]
GVFS_drive_hardisk_root_icon.patch
Comment 16 Nicolò Chieffo 2009-09-13 10:24:25 UTC
Created attachment 143096 [details] [review]
NAUTILUS_drive_hardisk_root_icon.patch

I have to test this
Comment 17 Nicolò Chieffo 2009-09-13 10:47:32 UTC
Created attachment 143097 [details] [review]
NAUTILUS_drive_hardisk_root_icon.patch

This one fixes a bug: missing icon in the placebar if the theme does not supply -root icon. That is because NautilusIconInfo didn't use fallbacks, so I added it.

tested works
Comment 18 Allan Day 2010-06-01 21:25:46 UTC
I've requested a specific icon for File System in bug 620239. What we still seem to be lacking are clear arguments why this change is necessary. Any comments? I'm marking this bug as NEEDINFO.
Comment 19 Allan Day 2010-06-01 21:50:30 UTC
*** Bug 561926 has been marked as a duplicate of this bug. ***
Comment 20 Tobias Mueller 2010-07-17 16:54:55 UTC
Nicolò, could you comment on the questions raised by Allan in comment #18?
Comment 21 Nicolò Chieffo 2010-07-17 17:37:42 UTC
I didn't understand the request. Why do you think it's not good to show the distribution icon in the filesystem?
Comment 22 Moo 2010-07-17 17:57:11 UTC
I want a distribution icon on the file system to increase usability.

I have many disk drives on the computer with multiple partitions.
Then it would increase usability to have an distribution icon in order for the user to quickly be able to visually recognize which file system it is that have the operating system installed and distinguish it from other file systems.

It would also be nice to have a Windows logo icon on the disk drive on any file system with Windows installed.
Comment 24 Tobias Mueller 2010-07-17 18:05:27 UTC
Reopening as per the last comments.
Comment 25 Cosimo Cecchi 2010-11-26 13:24:37 UTC
Review of attachment 143095 [details] [review]:

This should be filed as a bug against GVfs, not nautilus.
Comment 26 Cosimo Cecchi 2010-11-26 13:25:27 UTC
Review of attachment 143097 [details] [review]:

::: nautilus-2.27.92.old/src/nautilus-places-sidebar.c
@@ +437,3 @@
 	NautilusIconInfo *info;
 	
+	icon = g_themed_icon_new_with_default_fallbacks (name);

Not sure why you need to change the default NautilusIconInfo lookup function.
Comment 27 Cosimo Cecchi 2010-11-26 13:26:47 UTC
My preference for this is to use a distributor emblem composed over the file system icon, like Windows does.
It would be nice to have the distributor icon as a standard icon name probably.
Comment 28 Chris Wilson 2011-01-16 13:53:50 UTC
I think this bug should be reassigned to GVfs as per comment #25 as that is where these icons are hardcoded.
Comment 29 Timothy Arceri 2011-08-01 01:55:42 UTC
Currently Nautilus does its own calls to get the Icon (This can be seen by the patches). Changes currenlty need to be made to both Nautilus and GVfs and Nautilus also has the icons hardcoded. 

>Not sure why you need to change the default NautilusIconInfo lookup function.

As you can see be reading the comment trail, the call needs to be changes so that if the icon does not exist, then it falls back to the default filesystem icon rather than an ugly missing icon.
See the GTK doc for more details: http://developer.gnome.org/gio/2.28/GThemedIcon.html#g-themed-icon-new-with-default-fallbacks

Finally a new icon name of drive-harddisk-system has been created as seen by the depended bug https://bugzilla.gnome.org/show_bug.cgi?id=620239 and the case for this Icon change seems to now be generally accepted.

If I update the patch to the latest code, will the Nautilus developers apply it?
Comment 30 Timothy Arceri 2011-08-01 02:26:40 UTC
Ok, I have filed the bug report for the change needed for gvfs.
https://bugzilla.gnome.org/show_bug.cgi?id=655692

Also I now get the comment questioning why NautilusIcoInfo needed to be change.

The patch submitted commented:
This one fixes a bug: missing icon in the placebar if the theme does not supply
-root icon. That is because NautilusIconInfo didn't use fallbacks, so I added
it.

This should probably looked at to see if there is a better way to change.
Comment 31 Timothy Arceri 2011-08-22 09:21:20 UTC
I was about to post an updated patch for this bug, when I realised it is fixed in 3.1.4.

is set to NAUTILUS_ICON_FILESYSTEM = "drive-harddisk-system"

:)

Can someone please close this bug
Comment 32 Cosimo Cecchi 2011-08-22 13:35:46 UTC
Closing, thanks.