The default bug view has changed. See this FAQ.

[10.8] Mac dmg installer dialog doesn't look right

RESOLVED FIXED

Status

()

Firefox
Installer
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: marcia, Assigned: smichaud)

Tracking

(Blocks: 1 bug)

Trunk
x86
Mac OS X
Points:
---

Firefox Tracking Flags

(firefox13+ verified, firefox14+ verified)

Details

(Whiteboard: [qa+])

Attachments

(7 attachments)

(Reporter)

Description

5 years ago
Created attachment 606793 [details]
Screenshot of Beta dialog

Apple just released a new seed today, 12A154q. I noticed when downloading the beta that the dialog doesn't look quite right. Attaching a few screenshots.
(Reporter)

Comment 1

5 years ago
Checked Aurora and Nightly and the same thing happens.
(Reporter)

Comment 2

5 years ago
Now that there is a new Dev Preview I will check this again.
(Reporter)

Comment 3

5 years ago
Still happening in the latest Dev Preview, Build ID 12A193i when downloading the latest Firefox beta build.
(Assignee)

Comment 4

5 years ago
This looks really bad.  I'll see if I can fix it next week.
Assignee: nobody → smichaud
Blocks: 728102
(Assignee)

Comment 5

5 years ago
What causes this bug is that the minimum size for a "folder window"
has changed (increased) on OS X 10.8, so that it's now wider than the
background image used by our dmg installer's "folder window" (the
window that opens when you double-click on the dmg installer).

I'd hoped to force the window to display at a smaller than minimum
width.  And I actually accomplished this ... but the effect wasn't
permanent.

So it now looks like the only way to fix this bug is to increase the
width of the background image.

For the record, here's what I did:

1) Created and mounted a read-write DMG corresponding to FF 12's dmg:

   a) hdiutil convert "Firefox 12.0.dmg" -format UDRW -o stage1.dmg
   b) hdiutil resize -size 150m stage1.dmg
   c) hdiutil attach stage1.dmg -readwrite -autoopen

2) Ran the following AppleScript to correct the dmg folder window's
   size:

   tell application "Finder"
   tell window 1
       set current view to icon view
       set toolbar visible to false
       set bounds to {362, 95, 778, 390}
   end tell
   end tell

   Running this script changed the /Volumes/Firefox/.DS_Store file,
   and *might* have stored the new size and location information.  But
   you'll see in step 4 that this didn't help.

3) Detached the altered stage1.dmg and converted it back to readonly
   format:

   d) hdiutil detach /dev/disk1
   e) hdiutil convert stage1.dmg -format UDBZ -o stage2.dmg

4) Mounted stage2.dmg and observed that its folder window was too wide
   -- that it's width was the new Mountain Lion minimum width, and not
   the smaller width that I'd (presumably) caused to be written to
   .DS_Store in step 2.
(Assignee)

Comment 6

5 years ago
>   tell application "Finder"
>   tell window 1
>       set current view to icon view
>       set toolbar visible to false
>       set bounds to {362, 95, 778, 390}
>   end tell
>   end tell

I cut this down as follows and tried again.  Still no luck.

   tell application "Finder"
   tell window 1
       set bounds to {362, 95, 778, 390}
   end tell
   end tell

Updated

5 years ago
tracking-firefox14: --- → +
(Assignee)

Comment 7

5 years ago
Alex, it looks like you created the original background.png artwork for FF releases, way back in bug 320155.

Any chance you could widen it?

The same thing also needs to be done for the background.png artwork for Aurora and Nightly.  The files that need to be changed are here:

browser/branding/aurora/background.png
browser/branding/nightly/background.png
browser/branding/official/background.png

I'll get you an exact number of pixels that they need to be widened.  I suspect (but am still not sure) that the images need to be widened by half that number of pixels on each side.
(Assignee)

Comment 8

5 years ago
The "original" (and incorrect) bounds of the FF release dmg's folder window on OS X 10.8 are {362, 95, 799, 390}.

I got this by running the following AppleScript:

   tell application "Finder"
   tell window 1
       get bounds
   end tell
   end tell

The bounds of the correctly sized folder window (correctly sized by trial and error for the current background.png artwork) are:

{362, 95, 778, 390}

That's a difference of 21 pixels.  Awkward that it's an odd number.
(Assignee)

Comment 9

5 years ago
Created attachment 626609 [details]
"Official" background.png resized for Mountain Lion

Maybe this isn't as difficult as I first thought.

Here's a copy of the "official" background.png that I resized in Photoshop (CS5).  I added 22 pixels to the width, and told Photoshop to change the height proportionally.

I've tried it in an altered FF 12 dmg on Moutain Lion and it looks fine ... at least to me.  I'll also try it on other versions of OS X.

Alex Faaborg, let us know if you'd be willing to let us use this resized image.  Shortly I'll attach resized copies of the aurora and nightly background.png files.
Created attachment 626615 [details]
Aurora background.png resized for Mountain Lion
Created attachment 626618 [details]
Nightly background.png resized for Mountain Lion
Here's a copy of the altered FF 12 dmg package that I've been testing with (which contains my resized "official" background.png):

https://people.mozilla.com/~stmichaud/ResizedBackground/Firefox%2012.0-resized-background.dmg

It looks fine (to me) on OS X 10.5.8, 10.6.8, 10.7.4 and 10.8.
(In reply to Steven Michaud from comment #12)
> Here's a copy of the altered FF 12 dmg package that I've been testing with
> (which contains my resized "official" background.png):
> 
> https://people.mozilla.com/~stmichaud/ResizedBackground/Firefox%2012.0-
> resized-background.dmg
> 
> It looks fine (to me) on OS X 10.5.8, 10.6.8, 10.7.4 and 10.8.

Looks good to me on 10.7 as well. Tracking for FF13 (although this is not a blocker). If Steven agrees, this can make it into our final beta, going to build on Monday.
tracking-firefox13: --- → +
Created attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

Gavin, you reviewed my last patch that changed the appearance of our dmg package's window.  Could you review this one, too?

If you'd rather someone else do the review, please pass it along.

I've replaced the existing background.png files in the following directories with the resized copies that I've attached here.  I resized them in Photoshop (CS5), and had Photoshop increase the width by 22 pixels and the height proportionally.

In my tests they now look fine on all supported (and soon to be supported) versions of OS X (10.5.8 through 10.8).
Attachment #626668 - Flags: review?(gavin.sharp)
(Assignee)

Updated

5 years ago
Attachment #626668 - Attachment is patch: true
> Gavin, you reviewed my last patch that changed the appearance of our dmg package's
> window.

That was bug 714951.
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

thanks!
Attachment #626668 - Flags: review?(gavin.sharp) → review+
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

Though maybe shorlander (or martell, who IIRC created these originally?) might want to generate new images that aren't rescaled.
Attachment #626668 - Flags: ui-review?(shorlander)
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

Landed on mozilla-central:
http://hg.mozilla.org/mozilla-central/rev/8990746591f9

I'd like to get this into FF 13, which makes it very urgent.  So I've landed it on the trunk on the strength of Gavin's r+.  If shorlander or martell think we need to change these images again, we can address that in a new bug.
(Assignee)

Updated

5 years ago
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

Review of attachment 626668 [details] [diff] [review]:
-----------------------------------------------------------------

(In reply to Steven Michaud from comment #18)
> I'd like to get this into FF 13, which makes it very urgent.  So I've landed
> it on the trunk on the strength of Gavin's r+.  If shorlander or martell
> think we need to change these images again, we can address that in a new bug.

Works for me. I will talk to martell to track down the files for the background images.
Attachment #626668 - Flags: ui-review?(shorlander) → ui-review+
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

[Approval Request Comment]
Bug caused by (feature/regressing bug #): 
na
User impact if declined: 
The dmg package's window will look broken on OS X 10.8 (Mountain Lion)
Testing completed (on m-c, etc.): 
One successful set of tryserver builds, ongoing trunk build (successful so far)
Risk to taking this patch (and alternatives if risky): 
Small risk of visual glitches in dmg package window, but none seen so far on any version of OS X.
String or UUID changes made by this patch: 
none
Attachment #626668 - Flags: approval-mozilla-beta?
Attachment #626668 - Flags: approval-mozilla-aurora?
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

[Triage Comment]
Thanks for getting this together in time for our final beta Monday. Approved for all branches.
Attachment #626668 - Flags: approval-mozilla-beta?
Attachment #626668 - Flags: approval-mozilla-beta+
Attachment #626668 - Flags: approval-mozilla-aurora?
Attachment #626668 - Flags: approval-mozilla-aurora+
Comment on attachment 626668 [details] [diff] [review]
Resize background.png files to fit Mountain Lion dmg window's minimum size

Landed on aurora:
http://hg.mozilla.org/releases/mozilla-aurora/rev/6db75fba7c00

Landed on beta:
http://hg.mozilla.org/releases/mozilla-beta/rev/cecfcccccd9d
(Assignee)

Updated

5 years ago
status-firefox13: --- → fixed
status-firefox14: --- → fixed
Whiteboard: [qa+]

Comment 23

5 years ago
Created attachment 628317 [details]
screenshot

I've tested this on Mac 10.8 and as you can see from the attachment, there still are white borders on the sides.
What did you test?  I didn't see this with the build1 RC of FF 13.0b6, which I tested with on OS X 10.8 yesterday.

Comment 25

5 years ago
That's strange, today with the same build, FF 13b6, everything looks fine.
Only one narrow white vertical line remains on the left side.
Except from this, everything is looking ok.

Considering this, I'm changing the flag to verified on Firefox 13 beta 6 on MacOS X 10.8.

(In reply to Steven Michaud from comment #24)
> What did you test?  I didn't see this with the build1 RC of FF 13.0b6, which
> I tested with on OS X 10.8 yesterday.
status-firefox13: fixed → verified

Comment 26

5 years ago
I've retested the dialog on:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:14.0) Gecko/20100101 Firefox/14.0 beta 6 (build2)

and it's looking as expected, without the white borders.

Setting the flag to Verified.
status-firefox14: fixed → verified

Comment 27

5 years ago
Created attachment 633425 [details]
screenshot

I've retested this on other test machine with MacOS X 10.8 and it seems that the issue isn't gone.
The tests were made on:
Mozilla/5.0 (Macintosh; Intel Mac OS X 10.8; rv:13.0) Gecko/20100101 Firefox/13.0.1 and also on Firefox 13 and 14beta7 with the same results as in the attachament.
(In reply to comment #27)

I'll bet you also had mounted the dmg of an earlier version of Firefox (one without this patch).  Or that you'd recently had it mounted.

I myself saw this once.  I think it's an Apple bug.
You need to log in before you can comment on or make changes to this bug.