Closed Bug 1158856 Opened 6 years ago Closed 1 year ago
Add a favicon next to a tile's domain
Based on mocks here: https://bug1150228.bugzilla.mozilla.org/attachment.cgi?id=8592387
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.
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?
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.
Let's use browser favicon. We should try to minimize assets from clients as much as possible. If no favicon is available, show blank.
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.
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.
No longer blocks: Sprint_CS_S1
Iteration: 42.3 - Aug 10 → 43.1 - Aug 24
Points: --- → 2
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.