958399 - Use apple-touch-icon links as the home screen icon for homescreen shortcuts

Status: RESOLVED DUPLICATE of bug 826400

(Firefox for Android Graveyard :: General, defect, P5)

Description

[Neil Taylor]

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:26.0) Gecko/20100101 Firefox/26.0 (Beta/Release)
Build ID: 20131205075310

Steps to reproduce:

It's possible this bug is the same as 159241. I am not cognoscenti enough to be quite sure. 

The web site shootproof.com allows photographers to create an "App" which they download via their phone's browser and then "bookmark" the result and "save bookmark to home". This should create a home screen icon (as created on the shootproof site). 

Here is one example.... http://ntpix.shootproof.com/app/16537. To recreate the problem, enter that link in an Android browser and follow the instructions.

In several other browsers (Internet, Chrome, Dolphin)the process works, produces a shortcut icon on the home page with a wedding picture (from my shoot) as the icon, which links successfully to the web app containing the rest of my selected pics from that shoot. 


Actual results:

Everything worked except that it was saved to the home page without the custom icon. I tried several times with minor variations in sequence always with the same result.


Expected results:

The shortcut should have appeared with the icon I specified on Shootproof's site. It did in several other browsers I tried (using the same link).

Comment 1

[Aaron Train [:aaronmt]]

CC'ing rnewman because .. icons.

Comment 2

[Richard Newman [:rnewman]]

Thanks for the report, Neil. I'll take a look when I get a moment.

Comment 3

[Richard Newman [:rnewman]]

That page doesn't have a favicon specified, which is the usual trick Fennec uses to assign an icon to a webapp in this context.

When I bookmark the page, hit Edit, Add to home screen, I get the "globe" favicon as the homepage icon.

My guess, then, is that you're expecting the apple-touch-icon-precomposed `link` tag to be used instead, which is what Safari does on iOS, and some other browsers have aped (such is the browser ecosystem).

Shootproof ought to use the standard syntax as well as the Apple-specific one:

  <link rel="icon" sizes="196x196" href="..."/>

http://www.w3.org/html/wg/drafts/html/master/links.html#rel-icon


If that doesn't work, then this bug morphs into "support apple-touch-icon links", which is Bug 921014.


tracking-fennec?, because we might need to apply some pressure to get that bug shipped, so it's worth evaluating how much it matters.

Comment 4

[Richard Newman [:rnewman]]

This won't be actionable until the Gecko layer stuff is fixed. Unassigning.

Comment 5

[Wesley Johnston (:wesj)]

rnewman, not sure what bug you're talking about? We can do this if we want (for pages that are currently open) just by looking at meta-tags. We may need to do some jiggering to notify the Java UI about this UI.

I'd prefer to just restrict this to pages that are already open. Storing these in our favicon cache is out of scope.

Comment 6

[Richard Newman [:rnewman]]

(In reply to Wesley Johnston (:wesj) from comment #5)
> rnewman, not sure what bug you're talking about? 

Bug 921014, which is to do this for all of Gecko. (See deps.)

> I'd prefer to just restrict this to pages that are already open. Storing
> these in our favicon cache is out of scope.

We should also expand the "add to home screen" favicon guesser to try a-t-i well-known URIs.

Comment 7

[Karl Dubost💡 :karlcow]

this seems to be a duplicate of Bug 921014

Comment 8

[Richard Newman [:rnewman]]

Looks like dependency progress is underway!

Comment 9

[Brad Lassey [:blassey] (use needinfo?)]

filter on [mass-p5]

Comment 10

[:Margaret Leibovic]

This is being implemented in bug 826400.