Closed
Bug 339196
Opened 19 years ago
Closed 19 years ago
Cleaner wizard image for NSIS installer
Categories
(Firefox :: Installer, defect)
Tracking
()
RESOLVED
FIXED
Firefox 2 beta1
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
Details
(Keywords: fixed1.8.1)
Attachments
(9 obsolete files)
I threw the original image together in a minute for the trunk and have since cleaned it up. New image coming up shortly.
![]() |
Assignee | |
Comment 1•19 years ago
|
||
Mike, the image I checked in was done in a rush and I'd like to land this cleaned up version on the trunk. I used the png on the Minefield welcome page to create it.
Attachment #223272 -
Flags: ui-review?(beltzner)
![]() |
Assignee | |
Comment 2•19 years ago
|
||
Screenshots of before and after so you can compare the change.
Comment 3•19 years ago
|
||
Filed Bug 339204 for the garbled BMP-image (at least on my computer).
Comment 4•19 years ago
|
||
Comment on attachment 223272 [details]
new image
This looks a little bit downsampled to me. Let's do this right, and get Steven to deliver us a quick graphic.
Attachment #223272 -
Flags: ui-review?(beltzner) → ui-review-
Comment 5•19 years ago
|
||
Steven - can you generate some new BMP images for the NSIS installer? We'd be looking for three, I think:
- using Minefield logo
- using Bon Echo/unbranded logo
- using Firefox logo
Thanks.
Comment 6•19 years ago
|
||
Here's a cleaner installer graphic for Minefield. I'm not sure what the format/dimensions requiremetns are for the NSIS installer - let me know if anything else/different is needed.
Attachment #223272 -
Attachment is obsolete: true
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223312 -
Flags: review+
![]() |
Assignee | |
Updated•19 years ago
|
Assignee: robert.bugzilla → steven
![]() |
Assignee | |
Comment 7•19 years ago
|
||
Attachment #223273 -
Attachment is obsolete: true
Attachment #223464 -
Flags: ui-review?(steven)
Updated•19 years ago
|
Attachment #223464 -
Flags: ui-review?(steven) → ui-review+
![]() |
Assignee | |
Comment 8•19 years ago
|
||
Steven, could you create two more images that are 150px width by 57px height along the lines of the attachment for the wizard header for official, trunk, and branch?
![]() |
Assignee | |
Comment 9•19 years ago
|
||
Attachment #223466 -
Flags: ui-review?(steven)
Comment 10•19 years ago
|
||
(In reply to comment #8)
> Steven, could you create two more images that are 150px width by 57px height
> along the lines of the attachment for the wizard header for official, trunk,
> and branch?
Yup. What are the color/alpha limitations. Are we able to have something with full alpha-transparency? I noticed another bug about how this installer header doesn't match if the system colors are off-white.
Updated•19 years ago
|
Attachment #223466 -
Flags: ui-review?(steven) → ui-review+
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223312 -
Attachment description: Cleaner Minefield install graphic → Cleaner Minefield install graphic (checked in)
Attachment #223312 -
Attachment is obsolete: true
![]() |
Assignee | |
Comment 11•19 years ago
|
||
Comment on attachment 223464 [details]
official image (checked in)
Mike, need this on the branch.
Attachment #223464 -
Flags: approval-branch-1.8.1?(mconnor)
![]() |
Assignee | |
Comment 12•19 years ago
|
||
Comment on attachment 223466 [details]
branch image (checked in)
and this one too.
Attachment #223466 -
Flags: approval-branch-1.8.1?(mconnor)
![]() |
Assignee | |
Comment 13•19 years ago
|
||
Checked in the left watermark wizard images to trunk. I'll post the image color/alpha limitations for Steven as soon as I find them.
![]() |
Assignee | |
Comment 14•19 years ago
|
||
Steven, the control used is the standard Static control with the SS_BITMAP style. I've never tried to get them to support alpha transparency and don't know if it is possible. Unless I am mistaken the bug about the header you are referring to involves using high contrast and isn't affected by the header image since the image doesn't cover the entire header (e.g. it isn't used for the background of the text with NSIS).
Comment 15•19 years ago
|
||
Robert - thanks for looking into that. The bug I had in mind was bug #267829 - any thoughts on that?
Status: NEW → ASSIGNED
Updated•19 years ago
|
Attachment #223466 -
Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
Updated•19 years ago
|
Attachment #223464 -
Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
![]() |
Assignee | |
Comment 16•19 years ago
|
||
Steven - NSIS forces a white background for the entire header. I just checked out a couple of msi setups from MS and they do the same thing so I think that it is acceptable to do this. There is at least one high contrast bug as well.
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223464 -
Attachment description: official image → official image (checked in)
Attachment #223464 -
Attachment is obsolete: true
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223466 -
Attachment description: branch image → branch image (checked in)
Attachment #223466 -
Attachment is obsolete: true
![]() |
Assignee | |
Comment 17•19 years ago
|
||
I used the mozicon128 and reduced its size for the header... not sure if it is acceptable but I had some time to do the header images.
Attachment #223758 -
Flags: review?(steven)
![]() |
Assignee | |
Comment 18•19 years ago
|
||
Attachment #223760 -
Flags: review?(steven)
![]() |
Assignee | |
Comment 19•19 years ago
|
||
Attachment #223761 -
Flags: review?(steven)
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223465 -
Attachment is obsolete: true
Updated•19 years ago
|
Attachment #223758 -
Flags: review?(steven) → review+
Updated•19 years ago
|
Attachment #223760 -
Flags: review?(steven) → review+
Updated•19 years ago
|
Attachment #223761 -
Flags: review?(steven) → review+
Comment 20•19 years ago
|
||
These look fine Robert. Thanks.
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223758 -
Flags: approval-branch-1.8.1?(mconnor)
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223760 -
Flags: approval-branch-1.8.1?(mconnor)
![]() |
Assignee | |
Updated•19 years ago
|
Assignee: steven → robert.bugzilla
Status: ASSIGNED → NEW
![]() |
Assignee | |
Comment 21•19 years ago
|
||
Fixed on trunk.
Status: NEW → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•19 years ago
|
Attachment #223758 -
Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
Updated•19 years ago
|
Attachment #223760 -
Flags: approval-branch-1.8.1?(mconnor) → approval-branch-1.8.1+
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223761 -
Attachment description: trunk → trunk (checked in)
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223758 -
Attachment description: official header → official header (checked in)
Attachment #223758 -
Attachment is obsolete: true
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223760 -
Attachment description: branch → branch (checked in)
Attachment #223760 -
Attachment is obsolete: true
![]() |
Assignee | |
Updated•19 years ago
|
Attachment #223761 -
Attachment is obsolete: true
![]() |
Assignee | |
Updated•19 years ago
|
Keywords: fixed1.8.1
Target Milestone: --- → Firefox 2 beta1
You need to log in
before you can comment on or make changes to this bug.
Description
•