Small desktop icons displayed when Windows desktop shortcuts set to display large icons

NEW
Assigned to

Status

()

Firefox
General
5 years ago
4 years ago

People

(Reporter: kbrosnan, Assigned: shorlander)

Tracking

({regression})

unspecified
x86
Windows 7
regression
Points:
---

Firefox Tracking Flags

(firefox24 unaffected, firefox25 unaffected, firefox26- affected, firefox27- affected, firefox28- affected)

Details

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
Created attachment 785802 [details]
small-icons-desktop.png

I have several shortcuts on my desktop to common versions of Firefox. My Windows desktop is set to display large icons. Sometime in the last several days the icons are only displayed in the small size.

7-24-2013 build is good

Setting 26 affected as I expect the merge is in process.

Comment 1

5 years ago
Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/dbd7d55d64ff
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130730 Firefox/25.0 ID:20130730062253
Bad:
http://hg.mozilla.org/mozilla-central/rev/129ce98f4cb2
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130730 Firefox/25.0 ID:20130730100815
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=dbd7d55d64ff&tochange=129ce98f4cb2


Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/c2dc8dbb77ef
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130729 Firefox/25.0 ID:20130729151217
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/93ba277cb5c4
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130729 Firefox/25.0 ID:20130729154316
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=c2dc8dbb77ef&tochange=93ba277cb5c4


Regression window(fx)
Good:
http://hg.mozilla.org/integration/fx-team/rev/673fab39681d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130730 Firefox/25.0 ID:20130730094111
Bad:
http://hg.mozilla.org/integration/fx-team/rev/72240998c094
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:25.0) Gecko/20130730 Firefox/25.0 ID:20130730101312
Pushlog:
http://hg.mozilla.org/integration/fx-team/pushloghtml?fromchange=673fab39681d&tochange=72240998c094


Suspected: Bug 889417
Blocks: 889417
Keywords: regressionwindow-wanted
(Assignee)

Comment 2

5 years ago
Are you only seeing this with the Nightly icon?

Comment 3

5 years ago
Aurora 25 builds are now available to check, adding a needinfo on kbrosnan.
Flags: needinfo?(kbrosnan)

Updated

5 years ago
Keywords: qawanted
(Reporter)

Comment 4

5 years ago
I've been booted into Linux recently. Will boot into Windows when I get a chance.

Updated

5 years ago
Duplicate of this bug: 906259

Comment 6

5 years ago
This did not happen with the Aurora update to 25a2.
Flags: needinfo?(kbrosnan)
Keywords: qawanted
status-firefox25: affected → unaffected
tracking-firefox25: ? → ---
tracking-firefox26: ? → +
Dolske - tracking 26 but really this only affects Nightly and will just continue to do so until fixed - can you ask someone to look into this?
Flags: needinfo?(dolske)
Hmm. I don't see anything wrong with the firefox.ico for the Nightly branding -- it's got 4 sizes -- 256x256, 48x48, 32x32, and 16x16. Same as the official branding's firefox.ico. I vaguely remember something about the order of the .ico contents mattering, but both the old and new nightly icons have the first 4 icons sizes in the same order. (The old nightly icon also includes 256 and 16 colors versions, but that shouldn't be in play here.)

Opening up the Nightly firefox.exe in VisualStudio to look at its icon resources looks normal to me too.

Yet I can also reproduce this on my Win8 desktop (right click desktop, View --> Large Icons). Release looks ok (I still had FF22 installed with the old icon, but updating it to FF23 with the new icon is also ok.) I guess it's good that this appears to be a Nightly-only bug, but I'm still a bit nervous not understanding why.

I'm not really sure what would cause this. Maybe rstrong has a clue?
Flags: needinfo?(dolske) → needinfo?(robert.bugzilla)
Summary: Small icons displayed when Windows desktop shortcuts set to display large icons → Small desktop icons displayed when Windows desktop shortcuts set to display large icons
Created attachment 794977 [details]
When using Extra Large icons (lol)

This is even more comical in the File Explorer, when using View --> Extra Large Icons
Not sure what is going on yet but I definitely see a difference between firefox.ico when compared with aurora and release.

Stephen, can you take a look at the nightly firefox.ico and compare it to the aurora firefox.ico that you created in bug 889417.
Flags: needinfo?(robert.bugzilla) → needinfo?(shorlander)
One more datapoint -- if I browse my mozilla-central tree with File Explorer in "extra large icons" mode, it correctly shows firefox.ico for all the branding flavors. So it seems like something in Windows just gets unhappy once the .ico is embedded in the .exe.
Also, saving the existing nightly firefox.ico using IcoFX makes it display properly.
Created attachment 794982 [details]
firefox.ico saved using IcoFX

I'll leave it up to Stephen if he wants to use the program he used to create the other icons or this one.
(Assignee)

Comment 14

5 years ago
Created attachment 806319 [details] [diff] [review]
Fix Updated Branding icos - i01

Saved with another editor:
- Fixes broken large Nightly icon
- Adds low color icons

Tested this on Windows XP — 8.1 with the exception of Vista.
Attachment #806319 - Flags: review?(dolske)
Flags: needinfo?(shorlander)
Comment on attachment 806319 [details] [diff] [review]
Fix Updated Branding icos - i01

Does this other editor have any better options for the 256-color (8bit) icons? They've got some pretty severe banding, would be nice if it could do a decent job of dithering. The 16-color icons are especially loltastic, but that's a lost cause to begin with.

Do we really need the 256x256 versions for 256 and 16 color?
Attachment #806319 - Flags: review?(dolske) → review+
Stephen can you wrap this one up?
Assignee: nobody → shorlander
status-firefox27: --- → affected
status-firefox28: --- → affected
tracking-firefox27: --- → +
tracking-firefox28: --- → +
(Assignee)

Comment 17

4 years ago
> Does this other editor have any better options for the 256-color (8bit)
> icons? They've got some pretty severe banding, would be nice if it could do
> a decent job of dithering. The 16-color icons are especially loltastic, but
> that's a lost cause to begin with.

It does have a dithering option, however it didn't look nicer than the huge banding IMO.

> Do we really need the 256x256 versions for 256 and 16 color?

As far as I can tell Vista+ doesn't support 256 and 16 color as a user setting, but I am not really sure if there is a case where they can end up in that mode. E.g. Remote Desktop.
Do we want this uplifted on Monday for the second-to-last beta?  Please nominate if so, I'm assuming this is a low-risk branding fix.
Flags: needinfo?(shorlander)
Flags: needinfo?(dolske)
Since this doesn't seem to be concerning anyone enough to land to central and nominate for uplift I'm removing tracking here.  There must not be very many users impacted and we can take an uplift when ready if low risk.
tracking-firefox26: + → -
tracking-firefox27: + → -
tracking-firefox28: + → -
Flags: needinfo?(shorlander)
Flags: needinfo?(dolske)
You need to log in before you can comment on or make changes to this bug.