Turns out we shouldn't be using an svg icon, much less the same icon for all sizes. The built in svg library is a bit wonky*, especially as you get to older versions, so we need to switch over to fixed size PNGs instead. Tracked down our real icons with some help from others and fixed the 16x16 size to not be clipped up top. https://engineering.redhat.com/trac/subscription-manager/attachment/wiki/GUIRedesign/SubMan-icons.zip That includes our actual source, and pngs for 16x16, 22x22, 24x24, 32x32, and 48x48. There may be some package wizardry needed to ensure the icons end up where they need to be, I think jbowes knows what to do. * - this accounts for the missing dial and text when displaying it in RHEL 5.x and the text that runs outside the image in RHEL 6.x/Fedora.
Created attachment 604692 [details] Missing Needle
Created attachment 604693 [details] Runs outside of icon
I screwed up the 16x16 PNG in the first version. It still had the gray baseplate in the background. New, clean version is included in the updated zip: https://engineering.redhat.com/trac/subscription-manager/attachment/wiki/GUIRedesign/Sub-Man-icons.zip
If anyone is powerful enough to delete the first zip file, I would be appreciative. I was trying to overwrite and replace it, but I had an errant hyphen and now we have two zip files up on trac.
fixed in master at cbe8fbacadc409041f8c74fa18ef3287aa5dfcc3.
This request was evaluated by Red Hat Product Management for inclusion in a Red Hat Enterprise Linux release. Product Management has requested further review of this request by Red Hat Engineering, for potential inclusion in a Red Hat Enterprise Linux release for currently deployed products. This request is not yet committed for inclusion in a release.
Created attachment 607347 [details] 16x16 icon with background color Based on the rpms I installed today*, we're using the wrong 16x16 icon. I see a background on the icon. We should be using the icon out of this zip - https://engineering.redhat.com/trac/subscription-manager/attachment/wiki/GUIRedesign/Sub-Man-icons.zip the version in there has a transparent background. I can't delete the other zip off trac... * - I used rpms from http://hudson.rhq.lab.eng.bos.redhat.com:8080/hudson/view/Entitlement/job/subscription-manager.el5/1935/
in master at dc0fdddd62682f61a3f7b0b7f42420e9567554e2
Created attachment 607629 [details] Updated Icon with no background color 16x16 icon looks great now. Minor issue, but it looks like the dialogs we spawn haven't had their icon location updated, and are back to showing the default generic window icon. It'd be nice if those could be mapped to the new png. Dialogs like: System Preferences About Import Cert System Facts Configure Proxy System Registration
Opened a BZ about comment 9 for 6.4 - 855050.
Created attachment 614935 [details] icons good Testing in latest brew: subscription-manager-gui.x86_64 0:1.0.18-1.el5 Looks like the icons are now the updated ones and the backgrounds to them are transparent. Additionally they are now updated to png and are at these locations: /usr/share/icons/hicolor/16x16/apps/subscription-manager.png /usr/share/icons/hicolor/22x22/apps/subscription-manager.png /usr/share/icons/hicolor/24x24/apps/subscription-manager.png /usr/share/icons/hicolor/32x32/apps/subscription-manager.png /usr/share/icons/hicolor/48x48/apps/subscription-manager.png Marking VERIFIED.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. http://rhn.redhat.com/errata/RHBA-2013-0033.html