Closed Bug 473339 Opened 12 years ago Closed 12 years ago

Refresh disk.icns for OS X

Categories

(Firefox :: Installer, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 3.6a1

People

(Reporter: faaborg, Assigned: faaborg)

References

()

Details

(Keywords: fixed1.9.1, polish, Whiteboard: [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh][polish-p3])

Attachments

(1 file)

This bug is about updating the DMG icon we use on OS X for the official Firefox branding, design discussion is at http://blog.mozilla.com/faaborg/2008/12/16/new-os-x-icons-made-by-sofa/
This file should replace the file at /other-licenses/branding/firefox/disk.icns
Keywords: checkin-needed
Whiteboard: [polish-easy] [polish-visual] [icon-3.1]
Attachment #356676 - Flags: ui-review+
Alex - it'll make it easier for anyone checking this in if you make an hg patch for this.  You'll need to make sure you have git-style diffs enabled.
http://hg.mozilla.org/mozilla-central/rev/e55e91615f34

(In reply to comment #2)
> Alex - it'll make it easier for anyone checking this in if you make an hg patch
> for this.

+1
Keywords: checkin-needed
Target Milestone: --- → Firefox 3.2a1
Attachment #356676 - Flags: approval1.9.1?
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment on attachment 356676 [details]
New disk.icns icon
[Checkin: Comment 3]

a191=beltzner
Attachment #356676 - Flags: approval1.9.1? → approval1.9.1+
Keywords: checkin-needed
Assignee: nobody → faaborg
Whiteboard: [polish-easy] [polish-visual] [icon-3.1] → [needs 1.9.1 landing] [polish-easy] [polish-visual] [icon-3.1]
Attachment #356676 - Attachment description: New disk.icns icon → New disk.icns icon [Checkin: Comment 3]
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/4993c949f0a5
Whiteboard: [needs 1.9.1 landing] [polish-easy] [polish-visual] [icon-3.1] → [polish-easy] [polish-visual] [icon-3.1]
I'll need to remove the larger resolutions from this file so that we can get the file size down.  Odds are users are not going to be looking at this with coverflow.
Whiteboard: [polish-easy] [polish-visual] [icon-3.1] → [polish-easy] [polish-visual] [icon-3.1][icon-refresh]
(In reply to comment #6)
> I'll need to remove the larger resolutions from this file so that we can get
> the file size down.  Odds are users are not going to be looking at this with
> coverflow.
You should file a new bug for that since this is RESOLVED FIXED.
This bug's priority relative to the set of other polish bugs is:
P3 - Polish issue that is in a secondary interface, occasionally encountered, or is not easily identifiable.

Hopefully people will notice, the new icon looks great, but P3 since it is commonly viewed at 16x16
Whiteboard: [polish-easy] [polish-visual] [icon-3.1][icon-refresh] → [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh][polish-p3]
You need to log in before you can comment on or make changes to this bug.