Open Bug 1158856 Opened 5 years ago Updated 4 years ago

Add a favicon next to a tile's domain

Categories

(Firefox :: New Tab Page, defect)

defect
Not set
Points:
5

Tracking

()

Firefox 42
Iteration:
43.1 - Aug 24

People

(Reporter: emtwo, Assigned: mzhilyaev)

References

(Blocks 1 open bug)

Details

(Whiteboard: .?)

Blocks: 1140185
Iteration: --- → 40.3 - 11 May
Whiteboard: .?
Blocks: 1160372
No longer blocks: 1160372
No longer blocks: 1150228
Depends on: 1150228
Iteration: 40.3 - 11 May → 41.1 - May 25
Assignee: msamuel → nobody
Blocks: 1145418
No longer blocks: 1140185
Iteration: 41.1 - May 25 → ---
Could probably reuse what was done in bug 1163888 for unifiedcomplete to show the appropriate favicon for a host.
Depends on: 1163888
note that we only show existing icons, we don't fetch missing ones, also because with the new security policy regarding having a principal to load a favicon, we wouldn't know where to get the principal.
Blocks: 1172713
No longer blocks: 1145418
Blocks: 1192057
No longer blocks: 1172713
Assignee: nobody → mzhilyaev
Iteration: --- → 42.3 - Aug 10
Target Milestone: --- → Firefox 42
This needs a bit more clarification. Aaron's comment here (https://bugzilla.mozilla.org/show_bug.cgi?id=1188015#c16) may suggest that we may want to attach a partner supplied favicon to the tile distribution spec like other tile images.   

I was under impression that we should be using browser favicon cache and display favicon for any tile which page has one.  

So, need clarification on expected functionality: browser cached favicons for all tiles, or a partner supplied favicon attached to tile distribution, or a mixture of both?
Flags: needinfo?(kghim)
Flags: needinfo?(athornburgh)
Is it possible/feasible to have a mixture both?

I don't want to limit the use of favicons, so browser cached versions are fine.

My question to you is whether those cached versions would be "official" by default, or "best guess".

However, I think partners would like to opportunity to update/change their favicon for Firefox users.
Flags: needinfo?(athornburgh)
Let's use browser favicon. We should try to minimize assets from clients as much as possible. If no favicon is available, show blank.
Flags: needinfo?(kghim)
Blocks: Sprint_CS_S1
If we use browser favicon, then we are likely to show tiles ONLY for history tiles, because a user have visited landing URLs of history tiles already.  While the suggested or directory tiles URLs may miss a favicon, because a user is likely have not visited their landing pages. Which may result in a strange UX, where a user sees favicon for history tiles and sees none for suggested/directory tiles.

We could request a favicon for the landing URL host, but that would mean the site may learn about a user loading newtab with that tile being played, hence a privacy leak.

I recommend giving this nug a few more thought from product/UX perspective.
A proper fix would be adding a favicon to the distro of the tile.
Flags: needinfo?(kghim)
For History tiles, we'll get the favicon from the browser. 

For Suggested tiles, we can request a favicon file to load onto the distro.
Flags: needinfo?(kghim)
No longer blocks: Sprint_CS_S1
Iteration: 42.3 - Aug 10 → 43.1 - Aug 24
Points: --- → 2
Points: 2 → 5
Blocks: 1202820
Blocks: 1204936
No longer blocks: 1192057
Blocks: 1192057
No longer blocks: 1202820, 1204936
You need to log in before you can comment on or make changes to this bug.