netfs Network icon only shows at 32X32

[bbjimmy]

hrev50329

Install netfs from the Haiku repository.

Modify the netfs_mount script to work... copy the file out of bin to the Desktop, change the permissions so that one may write, open in Styled edit, add #!/bin/sh as line one and mark the file executable. Double-click the file. Now there is a Networkn icon on the Desktop or in the Disks, "/" folder. icons at 32x32

change to icon view and change the icon size to anything other than 32x32 and the Network icon either is gone or looks like a TV display that is not properly synced.

icons at 40x40

I do not know what component is at fault as I could not locate the icon.

[pulkomandy]

This is an "old style" BeOS bitmap icon. It is probably provided only at 32x32 and possibly 16x16. For other sizes, we need a vector icon instead.

[humdinger]

But we do have scaling for old-style bitmap icons... See for example /bin/CortexAddOnHost.

[bbjimmy]

Network disk icon

[bbjimmy]

another Network Disk icon

[bbjimmy]

another Network Disk icon

[bbjimmy]

We should use one of these:

icons attatched, sory for the third file, my Internet connection bonked out and I wasn't sure it was saved.

[bbjimmy]

The icon seems to be here: ​http://cgit.haiku-os.org/haiku/tree/src/kits/tracker/TrackerIcons.rdef listed as

resource(R_ShareIcon) #'ICON' array

can we just replace this with a vicon ?

[miqlas]

bbjimmy, could we get a HDD icon just with the globe? IT would be a bit better, i think.

[bbjimmy]

This one might be ok?

[humdinger]

WRT the icon, I don't think using a version of the harddisk icon (maybe with an overlay) is quite consistent. Harddisk + overlay is currently used to indicate local drives (boot drive, nighlty boot drive, BeOS drive, Zeta drive etc.). The network drive is normally just a shared folder. I committed an icon for NAS devices, which may fit the bill:

[humdinger]

Device_NAS

[bbjimmy]

I think the icon should have some indication that it is networked. The world is being used in Haiku to represent a Network or that a file is used on the network ... IE html files. This icon does not indicate that it has anything to do with the network.

[axeld]

I like it, and even if something more fitting in bbjimmy's opinion comes along, it's always only a single commit away :-)

[bbjimmy]

Now we need to add the icon here: ​http://cgit.haiku-os.org/haiku/tree/src/kits/tracker/TrackerIcons.rdef

[humdinger]

For some reason, replacing
resource(R_ShareIcon) #'ICON' array { -- bitmapdata -- } and
resource(R_MICN_ShareIcon) #'MICN' array { -- bitmapdata -- }
with
resource(R_ShareIcon) #'VICN' array { --vectordata-- }
in ​TrackerIcons.rdef (and removing R_MICN_ShareIcon = 1051 from ​TrackerIcons.h) doesn't do the trick.
I now get the - I think - query icon...

Besides: Shouldn't it be investigated why the bitmap scaling doesn't work in this circumstance?
Maybe jscipione can look into it? IIRC he implemented the bitmap scaling.

[pulkomandy]

The TrackerIcons are generic icons for when the device does not provide anything. The "query" icon is also used as the "generic volume" icon when nothing is available.

Getting the icon happens in kits/tracker/IconCache.cpp, which then requests it from kits/tracker/Bitmaps.cpp:144 (GetIconResource). This function does not attempt any scaling, and will just return an error if called with a size other than 16 or 32. It does handle vector icons, however, so I'm surprised that doesn't work.

[humdinger]

It may be that this shared-icon doesn't work for my NFS4 mounted share...
I uploaded the built Tracker+lib to my ​Dropbox. Maybe bbjimmy can give it a try to see if the new vector icon appears for him.

Before starting the Tracker from that package, you have to "launch_roster stop x-vnd.be-trak" to prevent the launch_damon from restarting the old system Tracker. Restart with "launch_roster start x-vnd.be-trak".

[bbjimmy]

This is slightly better, now it shows a vicon, just not the one expected:

[bbjimmy]

looking at libtracker.so with QuickRes, R-Shareicon looks like the query icon.

resource(R_ShareIcon) #'VICN' array { --vectordata-- } Are you sure you put the right vectordata in the source?

[humdinger]

I was sure that I pasted the correct rdef data, but now that you point it out... I managed to mess the copy&paste up. Damn! I'm committing the correct change momentarily... Thanks!

[axeld]

So can we close this now?

[humdinger]

It already is closed as duplicate. :)
There are other tickets dealing with the underlying issue of messed up scaled bitmap icons.