Ticket #1867 (closed bug: fixed)

Opened 9 months ago

Last modified 3 months ago

Icons for other volumes default to the file cabinet icon

Reported by: scottmc Owned by: axeld
Priority: normal Milestone: R1
Component: - General Version: R1 development
Cc: Blocked By:
Platform: All Blocking:

Description

Other BFS volumes default to the file cabinet icon and so do Audio CD and I assume Data CDs would as well. See the screenshot for #1866 for example.

Change History

Changed 3 months ago by axeld

  • status changed from new to closed
  • resolution set to fixed

I am not sure what Bruno tested, but it works fine over here on different machines. Fixed in r27001.

Changed 3 months ago by stippi

I would estimate that he set is Desktop icon size to something else than 16x16 or 32x32. If I read your patch correctly, you bail out early if the icon size is not one of those. This is different from the rest of the icon functions, where I check the color space of the icon. If it's 32 bit, it ignores the given icon_size and just uses the bitmap size. I can work on this today.

Changed 3 months ago by axeld

Actually, I just copied what was already there. But please remove that check; I would be all in favour of getting rid of that icon_which parameter everywhere, but for R1, I thought consistency would be the preferred way.

Note: See TracTickets for help on using tickets.