Closed
Bug 265826
Opened 20 years ago
Closed 20 years ago
update linux installer artwork
Categories
(Firefox :: Installer, defect)
Tracking
()
VERIFIED
FIXED
People
(Reporter: bryner, Assigned: bryner)
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file)
133.40 KB,
patch
|
vlad
:
review+
asa
:
approval-aviary+
|
Details | Diff | Splinter Review |
We should use the Firefox installer artwork that the Windows installer uses. In the process of working on this, I discovered that the new images are quite large as xpm's, and this would adversely affect the download size. So, I rewrote the image loading code in the installer such that we can use png's for the images. I also set the default window width to be the width of the header image to avoid clipping it on the right.
Assignee | ||
Comment 1•20 years ago
|
||
Make the installer artwork be per-product. Add unofficial and official versions of the header and watermark images, as png's.
Assignee | ||
Updated•20 years ago
|
Attachment #163267 -
Flags: review?(vladimir)
Comment on attachment 163267 [details] [diff] [review] patch r=vladimir@pobox.com
Attachment #163267 -
Flags: review?(vladimir)
Attachment #163267 -
Flags: review+
Attachment #163267 -
Flags: approval-aviary?
Assignee | ||
Comment 3•20 years ago
|
||
landed on branch (asa gave approval on irc) and trunk.
Comment 4•20 years ago
|
||
Your trunk checkin missed mozilla/toolkit/mozapps/installer/unix/wizard/Makefile.in, causing build bustage. Fixed.
Updated•20 years ago
|
Attachment #163267 -
Flags: approval-aviary? → approval-aviary+
Comment 6•20 years ago
|
||
hmmm this bug seems to have broken the icons in the Thunderbird windows installer. I am no longer seeing the header image in the wizard in today's build.
Comment 7•20 years ago
|
||
re-opening so we can fix the tbird bustage once RC1 is out the door.
Comment 8•20 years ago
|
||
re-opening so we can fix the tbird bustage once RC1 is out the door.
Status: VERIFIED → REOPENED
Resolution: FIXED → ---
Comment 9•20 years ago
|
||
brian, was the patch posted in this bug really the complete patch? I'm having a hard time resolving it against what actually got checked in which includes changes to thunderbird's installer but I don't see them here in the actual patch...Just trying to figure things out....
Comment 10•20 years ago
|
||
oops I think i put my comments in the wrong bug. I meant to be re-opening Bug #225281. sorry for the noise!
Status: REOPENED → RESOLVED
Closed: 20 years ago → 20 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Status: RESOLVED → VERIFIED
Updated•18 years ago
|
QA Contact: bugzilla → installer
You need to log in
before you can comment on or make changes to this bug.
Description
•