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 312095 - Wrong icon for network:///
Wrong icon for network:///
Status: RESOLVED FIXED
Product: nautilus
Classification: Core
Component: general
2.11.x
Other All
: Normal minor
: ---
Assigned To: Nautilus Maintainers
Nautilus Maintainers
Depends on: 167107
Blocks: 312134
 
 
Reported: 2005-07-30 23:21 UTC by Michael Monreal
Modified: 2005-08-29 09:22 UTC
See Also:
GNOME target: ---
GNOME version: 2.11/2.12


Attachments
Proposed prelimitary patch (736 bytes, patch)
2005-07-30 23:31 UTC, Christian Neumair
none Details | Review
Proposed patch (2.86 KB, patch)
2005-07-31 19:36 UTC, Christian Neumair
reviewed Details | Review

Description Michael Monreal 2005-07-30 23:21:37 UTC
Version details: nautilus 2.11.90
Distribution/Version: Ubuntu Breezy + Garnome

Nautilus uses a plain folder icon for the network:/// URI instead of the network
icon (other URIs like computer:///, burn:/// and trash:/// use their special
icons, too). This affects both pathbar and bookmarks.
Comment 1 Christian Neumair 2005-07-30 23:27:34 UTC
Thanks for pointing this out!
nautilus_icon_factory_get_icon_for_file lacks special-case code for this. I'd
like to get this fixed once the patch from bug 167107 is committed, since the
latter is quiet huge compared to this tiny code addition.
Comment 2 Christian Neumair 2005-07-30 23:31:32 UTC
Created attachment 49994 [details] [review]
Proposed prelimitary patch

I'm not sure whether this will still work after applying the patch from bug
167107, but this is the route to go :).
Comment 3 Michael Monreal 2005-07-31 12:51:24 UTC
Just rebuild with all three patches (nautilus-private, the 16px patch and this
one) applied. Builds fine and seems to work... 

But something I noticed before and is still true now: the special icons are not
always used in the places bar, i think only after the URI has been visited! For
example, if you 'killall nautilus' and have computer:///, network:///, burn:///
in the places bar, they show up as normal folders, clicking on them or typing
the uri  in the location bar changes the icon.
Comment 4 Christian Neumair 2005-07-31 13:36:35 UTC
Michael: I can reproduce this and will investigate it later - it is unrelated to
this problem, though. Feel free to open a new bug report :).
Comment 5 Michael Monreal 2005-07-31 16:10:13 UTC
The other bug is now filed as #312134 now btw
Comment 6 Christian Neumair 2005-07-31 19:36:44 UTC
Created attachment 50032 [details] [review]
Proposed patch

Thanks. I've just realized that I forgot smb:/// and that a combined patch
against this one and bug 312134 is a good idea.
Comment 7 Michael Monreal 2005-07-31 20:11:50 UTC
Just gave it a shot, again using the patch together with the ones from #167107,
seems to be fine.

Only thing I noticed, dunno if this is intended: smb:// displays the network
icon, navigating even further, like smb://workgroup or just smb://host again
displays the plain folder icon... shouldn't those use the network icon, too? And
what about the sftp:// protocol? Wasn't there some "network filesystem" icon
(folder with BNC  T connector or something like that) for this kind of stuff
(would fit for smb, too and I suppose nfs is in the same state as sftp, only
displaying a folder)?
Comment 8 Christian Neumair 2005-07-31 23:04:35 UTC
Michael: Thanks for your productive input. I'm sure we can do better in some
areas, but we still can investigate this later. Imho the "BNC T connector"
should only be used for volumes and shares, not their subfolders.
Comment 9 Christian Neumair 2005-08-05 23:50:20 UTC
I've submitted the attached patch to the nautilus mailing list [1] for review.

[1] http://mail.gnome.org/archives/nautilus-list/2005-August/msg00062.html
Comment 10 Michael Monreal 2005-08-23 09:18:20 UTC
Any news regarding this and bug 312134?
Comment 11 Christian Neumair 2005-08-29 09:22:01 UTC
A similar patch was committed, closing.