Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Icons with 96x96 or 512x512 icon size in manifest results in no icon generated on win 7 on app install

RESOLVED WONTFIX

Status

Firefox Graveyard
Web Apps
P3
normal
RESOLVED WONTFIX
5 years ago
2 years ago

People

(Reporter: jsmith, Unassigned)

Tracking

({dev-doc-complete})

15 Branch
x86_64
Windows 7
dev-doc-complete

Details

Attachments

(1 attachment)

(Reporter)

Description

5 years ago
Created attachment 621227 [details]
No Chrome Directory with Multiple Icons

Steps:

1. Install an application that contains references to icons that are (16x16, 32x32, 48x48, 64x64, 96x96, 128x128, 256x256, 512x512)

Expected:

The desktop shortcut and start menu items should contain an icon. %APPDATA% should also have a chrome directory with the appropriate icon to use for the desktop shortcut and start menu.

Actual:

No icon is generated. Checking %APPDATA% - Looks like no chrome directory was generated. The result - No icon for desktop shortcut or start menu item.

Sample manifest I used:

{
	"name": "Pipedrive Multi",
	"description": "Simple and powerful sales pipeline management tool",

	"launch_path": "/testcases/fancybox.html",

	"installs_allowed_from": ["*"],
	
	"icons": {
		"16": "/appinstall/_pipedrive_16x16.png",
		"32": "/appinstall/_pipedrive_32x32.png",
		"48": "/appinstall/_pipedrive_48x48.png",
		"64": "/appinstall/_pipedrive_64x64.png",
		"96": "/appinstall/_pipedrive_96x96.png",
		"128": "/appinstall/_pipedrive_128x128.png",
		"256": "/appinstall/_pipedrive_256x256.png",
		"512": "/appinstall/_pipedrive_512x512.png"
	},
	
	"developer":
	{
		"url": "http://quality.mozilla.org",
		"name": "Mozilla QA"
	}
}

Example Test App from Marketplace: https://marketplace.mozilla.org/en-US/app/pipedrive-crm/
(Reporter)

Comment 1

5 years ago
Random Question - What is our algorithm for choosing an icon from a list of icons?
(Reporter)

Comment 2

5 years ago
Tested also the same marketplace app on Mac - This does not happen on Mac. This is a windows-specific bug.
(Reporter)

Updated

5 years ago
Summary: Listing support for many different icons (16, 32, 48, 64, 96, 128, 256, 512) results in no icon generated on win 7 → Listing support for many different icons (16, 32, 48, 64, 96, 128, 256, 512) results in no icon generated on win 7 on app install
(Reporter)

Comment 3

5 years ago
Felipe explained to me that the largest icon is chosen when there are multiple icons. Also, this may be a problem with what icon sizes we support. I'll dig into this to figure out a matrix of icons that work and that do not work.
(Reporter)

Comment 4

5 years ago
Test Results:

- 16x16: Pass
- 32x32: Pass
- 48x48: Pass
- 64x64: Pass
- 96x96: Fail
- 128x128: Pass
- 256x256: Pass
- 512x512: Fail
Summary: Listing support for many different icons (16, 32, 48, 64, 96, 128, 256, 512) results in no icon generated on win 7 on app install → Icons with 96x96 or 512x512 icon size in manifest results in no icon generated on win 7 on app install
(Reporter)

Updated

5 years ago
Blocks: 731054
(Reporter)

Comment 5

5 years ago
We need to define what icon sizes we intend to support. Flagging dev-doc-needed.
Keywords: dev-doc-needed
Priority: -- → P3
(Reporter)

Updated

5 years ago
Target Milestone: --- → Firefox 15
(Reporter)

Updated

5 years ago
No longer blocks: 731054
Target Milestone: Firefox 15 → ---

Comment 6

5 years ago
Manifest doc is fixed.

https://developer.mozilla.org/en/Apps/Manifest#Fields
Keywords: dev-doc-needed → dev-doc-complete
(Reporter)

Updated

5 years ago
QA Contact: jsmith
(Reporter)

Updated

5 years ago
status-firefox16: --- → wontfix
Blocks: 1111077

Comment 7

3 years ago
Still an issue with Firefox 35 on both Windows 7 and 8. Linux and Mac has no issue. Confirmed with my own manifest and with a handful of manifests from the Marketplace.

All web app documentation everywhere claims 512px icons are required. Which would lead to a lot of iconless app installs for Windows users.
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.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
(Assignee)

Updated

2 years ago
Product: Firefox → Firefox Graveyard
You need to log in before you can comment on or make changes to this bug.