1303379 - Remove all uses of the old TopSites once AS is enabled for everyone

Status: RESOLVED FIXED

(Firefox for iOS :: Home screen, defect, P1)

Description

[Farhan Patel [:farhan]]

There are a few small hacks that I'm having to make in order to support both the old topSites and the new ActivityStream. This bug is to track the changes I'll need to make once AS is fully launched. 

- Remove TopSitesPanel.swift
- Remove ThumbnailCell.swift
- Remove default TopSite images in assets folder.
- Change metadata in DefaultSuggestedSites to use a simpler asset path for images.
- Make amazon/Wikipedia have white backgrounds.
- Remove color hack in TopSitesCell that changes black backgrounds to white ones.

Comment 1

[Justin D'Arcangelo [:justindarc]]

Farhan, is this something that should be done for v8.0?

Comment 2

[Farhan Patel [:farhan]]

Nope. We are enabling AS for 50 percent of the audience so we'll need the old Topsites.

Comment 3

[Farhan Patel [:farhan]]

Attachment: Pull Request

Comment 4

[Farhan Patel [:farhan]]

master https://github.com/mozilla-mobile/firefox-ios/commit/fa10eae3f278860424dde87965f3a107a62b85ed

Comment 5

[Stefan Arentz | :st3fan | ⏰ EST | he/him]

Uplifted to v8.x