Closed
Bug 473343
Opened 15 years ago
Closed 15 years ago
Refresh document.ico
Categories
(Firefox :: General, defect)
Tracking
()
VERIFIED
FIXED
Firefox 3.6a1
People
(Reporter: faaborg, Assigned: jboriss)
References
Details
(Keywords: polish, verified1.9.1, Whiteboard: [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh][polish-p2])
Attachments
(2 files, 6 obsolete files)
24.50 KB,
image/png
|
Details | |
288.68 KB,
application/octet-stream
|
beltzner
:
ui-review+
|
Details |
This bug is for refreshing the icon we use for HTML (and other) documents on Windows XP.
Reporter | ||
Updated•15 years ago
|
Whiteboard: [polish-easy] [polish-visual] [icon-3.1]
Reporter | ||
Comment 1•15 years ago
|
||
This file should replace the file at /other-licenses/branding/firefox/document.ico
Attachment #356683 -
Flags: ui-review+
Reporter | ||
Updated•15 years ago
|
Keywords: checkin-needed
Updated•15 years ago
|
OS: Mac OS X → Windows XP
Comment 2•15 years ago
|
||
Comment on attachment 356683 [details]
New document.ico for Windows XP
This seems to be an 48*48px icon with a lot of padding, making the image 128*128px big. I don't think that's intentional, but please correct me if I'm wrong.
Attachment #356683 -
Flags: review-
Updated•15 years ago
|
Keywords: checkin-needed
Reporter | ||
Comment 3•15 years ago
|
||
>This seems to be an 48*48px icon with a lot of padding, making the image
>128*128px big. I don't think that's intentional, but please correct me if I'm
>wrong.
I think that's because icons can not be larger than 48x48 on XP, but I'll make sure the icon file was generated correctly (I believe they are also missing 16 color versions, for all of those people still living in the early 90s).
Comment 4•15 years ago
|
||
(In reply to comment #1) > Created an attachment (id=356683) [details] > New document.ico for Windows XP > > This file should replace the file at > /other-licenses/branding/firefox/document.ico If this is only for XP installer changes will be required to set the icon differently between XP and other versions of Windows and it will need to be added along with the existing icon.
Reporter | ||
Updated•15 years ago
|
Whiteboard: [polish-easy] [polish-visual] [icon-3.1] → [polish-easy] [polish-visual] [icon-3.1][icon-refresh]
Reporter | ||
Updated•15 years ago
|
Whiteboard: [polish-easy] [polish-visual] [icon-3.1][icon-refresh] → [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh]
Reporter | ||
Updated•15 years ago
|
Summary: Refresh document.ico on XP → Refresh document.ico
Assignee | ||
Comment 5•15 years ago
|
||
Are the 8-bit and 4-bit icons missing/needed? I tried to generate some using Iconbuilder, which apparently didn't work - but if they are needed, either I can find some software to generate them, or perhaps someone can generate some based on the regular versions and then I can clean them up. Please let me know, thanks.
Assignee | ||
Comment 6•15 years ago
|
||
I attached what should be the leftover icons, in a few formats: - Vista 48x48, 32x32, and 16x16, in 8-bit and 4-bit, as PNGs and (attempted) icos (I was getting unknown errors, and this file may not work) - Photoshop files with the above That should be every needed document icon, so if anything else is needed please ping me or comment here.
Assignee | ||
Comment 7•15 years ago
|
||
Remaining documnet icons: - 48x48 OSX icon - Photoshop & iso file with the above
Reporter | ||
Comment 8•15 years ago
|
||
Comment on attachment 382367 [details]
Remaining documnet icons: 8-bit and 4-bit Vista
Need a single .ico file for windows.
Attachment #382367 -
Flags: ui-review-
Reporter | ||
Comment 9•15 years ago
|
||
Comment on attachment 382369 [details] Remaining document icons: 48x48 OSX OS X files need to be grouped into a single .icns file and posted to bug 473341
Attachment #382369 -
Flags: ui-review-
Reporter | ||
Comment 10•15 years ago
|
||
Sorry my mistake, we are now using bug 496734 for the .icns OS X icon
Assignee | ||
Comment 11•15 years ago
|
||
Attaching .ico for Vista
Assignee | ||
Comment 12•15 years ago
|
||
Attaching .icns for OSX
Reporter | ||
Comment 13•15 years ago
|
||
Requesting blocking: the old Firefox logo needs to get updated now that we have deployed the other branding images. Note that the files attached will break on Windows 2000, I'll work on new ones.
Flags: blocking-firefox3.5?
Reporter | ||
Comment 14•15 years ago
|
||
Comment on attachment 382434 [details]
.ico for Vista
need to remove the 128x128 for Vista or we break on windows 2000
Attachment #382434 -
Flags: ui-review-
Reporter | ||
Updated•15 years ago
|
Attachment #382435 -
Attachment is obsolete: true
Updated•15 years ago
|
Flags: blocking-firefox3.5? → blocking-firefox3.5+
Reporter | ||
Updated•15 years ago
|
Attachment #356683 -
Attachment is obsolete: true
Reporter | ||
Updated•15 years ago
|
Attachment #382367 -
Attachment is obsolete: true
Reporter | ||
Updated•15 years ago
|
Attachment #382369 -
Attachment is obsolete: true
Reporter | ||
Updated•15 years ago
|
Attachment #382434 -
Attachment is obsolete: true
Reporter | ||
Comment 15•15 years ago
|
||
I tested the .ico file out on XP, Vista and 2000. Similar to bug 497333, the inclusion of a 128x128 resolution caused the file not to render at all on 2000. Details are here: https://bugzilla.mozilla.org/show_bug.cgi?id=497333#c21
Reporter | ||
Comment 16•15 years ago
|
||
Boriss: how are you doing the image resize for these? The pixel hinting seems a little off. To the left in this image is a smaller version of the Firefox icon I created for linux. The process I used was: -started from one size up (32x32) -used "bicubic sharper" for the resampling -tried to keep the canvas sizes even numbers, so 1 pixel doesn't get split across two We may also want to cut the shadows on the logo for everything but the 256x256, since we already have a nice gradient on the page itself.
Updated•15 years ago
|
Assignee: nobody → jboriss
Assignee | ||
Comment 17•15 years ago
|
||
Attaching .ico file without the 128x128 icon. This doesn't address faaborg's last comment, which I'll go through now, but at least there's now one that doesn't break the universe.
Comment 18•15 years ago
|
||
This is what that .ico file looks like in context on Windows 7: http://people.mozilla.org/~beltzner/images/document-in-context.png
Comment 19•15 years ago
|
||
At various sizes: http://people.mozilla.org/~beltzner/images/document-in-context-large-icons.png http://people.mozilla.org/~beltzner/images/document-in-context-medium-icons.png http://people.mozilla.org/~beltzner/images/document-in-context-small-icons.png
Assignee | ||
Comment 20•15 years ago
|
||
Adding a new version of the Vista icon without the 128x128 icon - reduced shadows per faaborg's comment.
Attachment #382763 -
Attachment is obsolete: true
Updated•15 years ago
|
Attachment #382791 -
Flags: ui-review+
Comment 21•15 years ago
|
||
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/d0720cea747b http://hg.mozilla.org/mozilla-central/rev/7eecd419288c
Comment 22•15 years ago
|
||
Verified fixed on trunk (code checkin) and 1.9.1 with Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.1) Gecko/20090612 Firefox/3.5 (.NET CLR 3.5.30729) ID:20090612072440
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
Target Milestone: --- → Firefox 3.6a1
Reporter | ||
Comment 23•15 years ago
|
||
This bug's priority relative to the set of other polish bugs is: P2 - Polish issue that is in a secondary interface, occasionally encountered, and is easily identifiable.
Whiteboard: [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh] → [polish-easy] [polish-visual] [icon-shiretoko][icon-refresh][polish-p2]
You need to log in
before you can comment on or make changes to this bug.
Description
•