Closed Bug 871806 Opened 11 years ago Closed 11 years ago

Work - Preloaded bookmarks have no color or favicon on first browser load

Categories

(Firefox for Metro Graveyard :: Bookmarks, defect)

x86_64
Windows 8.1
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Firefox 27

People

(Reporter: bbondy, Assigned: rsilveira)

References

Details

(Whiteboard: feature=work [preview][blocked])

Attachments

(1 file)

For new users, the first time they open the browser, they will see some predefined bookmarks, but they have neither color nor favicons.

They'll continue to see it in that state until they load the pages.

I think we should have color and favicons preloaded for the default preloaded sites, so people have a good first use experience.
Blocks: 791706
No longer blocks: metrov1defect&change
Summary: Preloaded bookmarks have no color or favicon on first browser load → Work - Preloaded bookmarks have no color or favicon on first browser load
Whiteboard: feature=work
Whiteboard: feature=work → feature=work [preview]
No longer blocks: 791706
Whiteboard: feature=work [preview] → feature=work [preview][blocked]
Attached patch 871806.patchSplinter Review
Added a data uri with the proper icon from http://mozorg.cdn.mozilla.net/media/img/favicon.ico.

We only have 2 items initially and they both have the same black and white icon where black is the dominant color for the tile color strip. Not much of a palette.
Assignee: nobody → rsilveira
Status: NEW → ASSIGNED
Attachment #813205 - Flags: review?(mbrubeck)
Comment on attachment 813205 [details] [diff] [review]
871806.patch

Review of attachment 813205 [details] [diff] [review]:
-----------------------------------------------------------------

Yeah, hopefully soon we can choose some more default bookmarks from sites with nicer favicons. :)
Attachment #813205 - Flags: review?(mbrubeck) → review+
https://hg.mozilla.org/mozilla-central/rev/dd50830b2dcd
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 27
OS: Windows 8 Metro → Windows 8.1
You need to log in before you can comment on or make changes to this bug.