Closed
Bug 382665
Opened 17 years ago
Closed 17 years ago
Use a better icon for the crash reporter on OS X
Categories
(Toolkit :: Crash Reporting, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: ted, Assigned: ted)
Details
Attachments
(1 file)
1.04 KB,
patch
|
mark
:
review+
|
Details | Diff | Splinter Review |
The Crash Reporter client currently uses some generic looking icon on OS X. We should use something better. I borrowed the updater icon on Windows, maybe we can do the same here?
Comment 1•17 years ago
|
||
To make an icon for an application, you need to create an icns file. There is an application that comes with the Developer Tools called Icon Composer. Use that to create the icns file. Make that icns file appear in the Resources/ directory of your built application, and make sure it has the same name as your application, and you should be good to go, if I'm remembering all of this correctly.
updater's icon is better than the generic app icon, and roughly the right motif, so if there's not a designer free to do better, that would placate me. (In reply to comment #1) > Make that icns file appear in the Resources/ directory of your built > application, and make sure it has the same name as your application, and you > should be good to go, if I'm remembering all of this correctly. The icon filename needs to be specified as the CFBundleIconFile in the app's Info.plist (the field is present, but currently blank, in crashreporter's Info.plist).
Assignee | ||
Comment 3•17 years ago
|
||
This just copies the updater icon, like I did on Windows.
Comment 4•17 years ago
|
||
Comment on attachment 268111 [details] [diff] [review] copy updater.icns The binary file isn't included in the diff, but I looked at toolkit/mozapps/update/src/updater/macbuild/Contents/Resources/updater.icns.
Attachment #268111 -
Flags: review?(mark) → review+
Assignee | ||
Comment 5•17 years ago
|
||
Yep, that's what I copied. It's generic enough that I think it will be ok. Checked in.
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•