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 567254 - Wrong icon for folder bookmarks
Wrong icon for folder bookmarks
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: Sidebar
2.24.x
Other Linux
: Normal minor
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
: 571209 (view as bug list)
Depends on:
Blocks: 539286
 
 
Reported: 2009-01-10 14:38 UTC by Krzysztof Kosiński
Modified: 2009-04-27 16:35 UTC
See Also:
GNOME target: ---
GNOME version: 2.23/2.24


Attachments
Icon fallback order patch (2.78 KB, patch)
2009-01-10 14:40 UTC, Krzysztof Kosiński
none Details | Review
Icon fallback order patch v2 (3.48 KB, patch)
2009-01-10 15:02 UTC, Krzysztof Kosiński
committed Details | Review

Description Krzysztof Kosiński 2009-01-10 14:38:17 UTC
+++ This bug was initially created as a clone of Bug #539286 +++

The bookmarks inside the places menu seem to be using the wrong icon when a theme like mist (themes inheriting the gnome theme) is used. In this case, bookmarks for special URIs like ssh:/// and obex:/// show the theme's folder icon while file:/// bookmarks only show the gnome folder.

This is caused by a wrong fallback order of icons assigned to folders. Icon names like folder-drag-accept, folder-open, folder-visiting and text-x-preview should be prepended to the list of icon names, rather than appended. This way, the inode/directory being present in the theme doesn't override the aforementioned special icons.
Comment 1 Krzysztof Kosiński 2009-01-10 14:40:56 UTC
Created attachment 126171 [details] [review]
Icon fallback order patch

This patch places appends icons like folder-drag-accept to the NautilusFile's icon rather than appending it, causing the correct behavior or treating the "inode-directory" icon as equivalent to "folder", and not as an override of all folder-* icons.
Comment 2 Krzysztof Kosiński 2009-01-10 15:02:39 UTC
Created attachment 126173 [details] [review]
Icon fallback order patch v2

Reworked patch that also makes "folder" override "inode-directory" to restore freedesktop.org icon naming compliance.
Comment 3 Baptiste Mille-Mathias 2009-01-11 10:09:13 UTC
Krzysztof,

Don't forget to send your patch to nautilus mailing-list
Comment 4 Victor 2009-01-18 20:49:42 UTC
I just browsed the nautilus mailing list and saw the patch posted there. No one seems to have replied to the thread and I haven't seen any comments regarding the bug in the trunk revisions changelog, will we get to see a resolution for this issue before a future stable release?
Comment 5 A. Walton 2009-02-11 08:11:41 UTC
*** Bug 571209 has been marked as a duplicate of this bug. ***
Comment 6 dedanna1029 2009-02-15 21:09:40 UTC
I am currently experiencing this issue in fully updated Fedora 10. When I click "Home" or "Documents" or whatever folder in the Places menu in Gnome, it opens with Easytag, rather than Nautilus. I was also experiencing the issue in Mandriva Cooker in Sept. 2008 (see https://qa.mandriva.com/show_bug.cgi?id=44350 ), but was able to at least semi-resolve it with the following:

rm -rf /usr/share/icons/gnome/scalable/places/inode-directory.svg
rm -rf /usr/share/icons/gnome/32x32/places/inode-directory.png
rm -rf /usr/share/icons/gnome/24x24/places/inode-directory.png
rm -rf /usr/share/icons/gnome/22x22/places/inode-directory.png
rm -rf /usr/share/icons/gnome/16x16/places/inode-directory.png
rm -rf /usr/share/icons/gnome/icon-theme.cache
gtk-update-icon-cache -qf /usr/share/icons/gnome


This has not worked for Fedora 10 though, sadly. :\
Comment 7 Cosimo Cecchi 2009-04-27 16:35:02 UTC
Thanks, committed to master.

commit 48ed43f32668e5615ac64a3de55ec7ab2f254ef7
Author: Cosimo Cecchi <cosimoc@gnome.org>
Date:   Mon Apr 27 18:34:18 2009 +0200

    Reverse the order for folder icons overrides
    
    Use the correct order for folder icons overrides, so that we can
    keep track of things like "folder-visiting" or "folder-drag-accept".
    Patch by Krzysztof Kosiński (#567254).