No default bookmarks in Kidfox

RESOLVED FIXED in Firefox 42

Status

()

defect
RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: ally, Assigned: ally)

Tracking

unspecified
Firefox 42
Points:
---

Firefox Tracking Flags

(firefox42 fixed)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Updated

4 years ago
Assignee: nobody → ally
(Assignee)

Comment 1

4 years ago
This probably needs to go on the tracking sheet, under the "no lies" rule for mvp, since it includes marketplace & amo.

Luckily, I already did this work trying to solve the tiles issue. :)
Flags: needinfo?(jchaulk)
(Assignee)

Comment 2

4 years ago
Attachment #8641255 - Flags: review?(s.kaspari)
Comment on attachment 8641255 [details] [diff] [review]
Kidfox-NoBookmarks

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

LGTM.

As soon as we want to ship this as a partner distribution we might want to revise but this works for now.
Attachment #8641255 - Flags: review?(s.kaspari) → review+
(Assignee)

Comment 4

4 years ago
(In reply to Sebastian Kaspari (:sebastian) from comment #3)
> Comment on attachment 8641255 [details] [diff] [review]
> Kidfox-NoBookmarks
> 
> Review of attachment 8641255 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> LGTM.
> 
> As soon as we want to ship this as a partner distribution we might want to
> revise but this works for now.

oh, totally. Though i think there's already a flow for that. I bet it looks a lot like the topsites tiles :/
https://hg.mozilla.org/mozilla-central/rev/4cb44bbf5e25
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 42
Please make sure bugs block the right meta bugs! I thought that bug 1189790 was the bug that was handling this.
Blocks: kidfox-v1
(Assignee)

Updated

4 years ago
Flags: needinfo?(jchaulk)
You need to log in before you can comment on or make changes to this bug.