I notice that many images in the modern theme (didn't check classic) are transparent, and need not be. Transparency imposes significant runtime cost, especially for images that must be tiled (at least on Mac). Some examples: mozilla/themes/modern/navigator/icons/btn1.gif mozilla/themes/modern/global/toolbar/tb-mid.gif (gets tiled!) also mozilla/themes/modern/global/preview.gif We should probably make a pass through all skin images and ensure that transparency is off when possible.
Assignee: hewitt → shliang
triage approving performance win
Status: NEW → ASSIGNED
Target Milestone: --- → mozilla0.9.9
i looked at those images in photoshop and they are not transparent
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → WORKSFORME
The GIFs have transparency.
Status: RESOLVED → REOPENED
Resolution: WORKSFORME → ---
Note: I used the GIF Analyzer, from <ftp://ftp.totallyhip.com/products/GIFUtilityPack/GIFUtilityPack10FAT.bin> to look at the image. It shows that the 'btn1.gif' is transparent, with transparency index 235. So even if PhotoShop shows no transparency, however you export to .gif is adding transparency, needlessly.
Created attachment 70167 [details] turning transparency off for those 3 images going to go through other images now
be careful - some images need to be transparent as far as I know.
Created attachment 70197 [details] classic win preview.gif doesn't look like anything else in modern or classic has transparency if it doesn't need it
sr=hewitt on all the images
r=andreww for images listed in this bug.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago → 17 years ago
Resolution: --- → FIXED
Verified on all platforms (commercial build: 2002-02-25-08-trunk)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.