742639 - Write all relevant icon sizes to the .ico files generated for natively-installed webapps

Status: RESOLVED WONTFIX

(Firefox Graveyard :: Web Apps, enhancement, P3)

Description

[Mark Hammond [:markh] [:mhammond]]

It seems only the larges icon specified by the webapp is embedded in the executable.  We should arrange to embed *all* images into the same "icon group" using the images native resolution.  We should also look at embedding in png format where possible to support windows 7 a little better (ie, you should then get variable transparency), then let Windows itself choose the icon to use based on the machine configuration and where the icon is to be used.

Comment 2

[Tim Abraldes [:TimAbraldes] [:tabraldes]]

We don't currently (and may never) embed icons in the webapp EXEs (see bug 745988), but we should definitely write all relevantly-sized icons to the .ico that is generated for each natively-installed webapp.  This will allow Windows to show (for example) a small icon for the system menu (in the top-left corner of the app), a larger icon for shortcuts and the taskbar, and an even larger icon in other cases.

Comment 3

[Brian R. Bondy [:bbondy]]

Although not webapp specific, I think this is a dupe of bug 419588.  Maybe we can add (Needed for webapps) to bug 419588 and close this?

Comment 4

[Jason Smith [:jsmith]]

(In reply to Brian R. Bondy [:bbondy] from comment #3)
> Although not webapp specific, I think this is a dupe of bug 419588.  Maybe
> we can add (Needed for webapps) to bug 419588 and close this?

Sounds like that bug is a dependency more so I think rather than a dup, as there's probably something we need to do on the web runtime side.

Comment 5

[Brian R. Bondy [:bbondy]]

Sounds good, thanks!

Comment 7

[Caspy7]

The blocking bug 419588 has now been fixed.
Fixing this will help make web apps first class citizens on the desktop.

Comment 8

[Myk Melez [:myk] [@mykmelez]]

Per bug 1238079, we're going to disable the desktop web runtime and remove it
from the codebase, so we won't fix these bugs in the integration between Firefox and the runtime.