Skip to content
This repository has been archived by the owner. It is now read-only.

Update Tippy Top with scraped icons #3149

Closed
Mardak opened this issue Aug 10, 2017 · 1 comment
Closed

Update Tippy Top with scraped icons #3149

Mardak opened this issue Aug 10, 2017 · 1 comment

Comments

@Mardak
Copy link
Member

@Mardak Mardak commented Aug 10, 2017

When tippy top has allowed icons, we'll want to use those

@tspurway
Copy link
Contributor

@tspurway tspurway commented Aug 30, 2017

@bryanbell We need to take another pass at the Tippytop icons for default Top Sites. We need to make two changes:

  1. use hi-res icons where available. Let's figure out which sites actually have decent icons (from the list of default top sites) and use them where available

  2. when we have to use screenshots, let's make sure that the favicon for that site is also put in the top-left hand corner of the screenshot icon

@tspurway tspurway assigned tspurway and unassigned rlr Sep 11, 2017
@rlr rlr assigned rlr and unassigned tspurway Sep 12, 2017
rlr added a commit to rlr/activity-streams that referenced this issue Sep 12, 2017
rlr added a commit to rlr/activity-streams that referenced this issue Sep 12, 2017
@rlr rlr closed this in #3435 Sep 12, 2017
rlr added a commit that referenced this issue Sep 12, 2017
fix (systemaddon) : #3149 icons for default sites
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
5 participants