389143 - thunderbird/icons/default.xpm mozicon50.xpm icons are bad - icon picture is much smaller than icon size on screen

Status: RESOLVED WORKSFORME

(Thunderbird :: General, defect)

Description

[Zbigniew Luszpinski]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; pl-PL; rv:1.8.1.5) Gecko/20070721 SeaMonkey/1.1.3
Build Identifier: version 2.0.0.5 (20070721)

In thunderbird/icons directory there are two files:
default.xpm
mozicon50.xpm
The screen size of icon is larger than picture of it. Because of this bug the icon  when used on desktop/menu is visible smaller than others.

When you open these 2 icons in any icon editor or paint program you will see small picture in the center and thick transparent border.

Reproducible: Always

Steps to Reproduce:
1. Use any paint program or icon editor to open these files. Such program must show transparent background as pattern to make the bug visible. I used to verify this: KIconEdit and gimp.

Actual Results:  
The default.xpm and mozicon50.xpm both have 48x48 icon size.
The real size of pictures in these icons are 31x32 and 29x31.
Because of this disproportion when these icons are used they look much smaller than others having the same standard 48x48 size. When in menu smaller icons are used both these icons are scaled down together with transparent border so they still look smaller than other icons.

Expected Results:  
The default.xpm and mozicon50.xpm both have 48x48 icon size and picture content should fill all drawing area not leaving any empty border around. The edges of picture should stick to borders of icon drawing frame.

I checked Firefox. FF2 Has the similar firefox/icons directory with the same icons names: default.xpm and mozicon50.xpm but of course with FF pictures. They are well done and drawn with beautiful perfection. Use them as a model when giving right picture size when redrawing Thunderbird icons.

Comment 1

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

Magnus, is this addressed with the recent TB Linux icon fixes?

Comment 2

[Magnus Melin [:mkmelin]]

Yeah this was fixed recently.