Hey :cmuntean. Thanks for sharing the results. My comments are inlined: (In reply to Cosmin Muntean [:cmuntean], Ecosystem QA from comment #5) > I have verified this issue using the latest top_picks.json and here are the results: > 1. "researchgate" and "glassdoor" no longer have an icon, so I am not sure if the issue is fixed for them. This is expected as per the latest changes. We have a minimum resolution criteria in place for favicons. Favicons for Researchgate don't satisfy it and Glassdoor don't have a favicon with correct mime type. > 2. For politico the issue seems to be fixed and the icon is correctly displayed. Great. > 3. I have found another result that has a broken favicon: **xfinity**. @Abhishek could you please take a look over this? Looking into this one.
Bug 1830535 Comment 6 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
Hey :cmuntean. Thanks for sharing the results. My comments are inlined: (In reply to Cosmin Muntean [:cmuntean], Ecosystem QA from comment #5) > I have verified this issue using the latest top_picks.json and here are the results: > 1. "researchgate" and "glassdoor" no longer have an icon, so I am not sure if the issue is fixed for them. This is expected as per the latest changes. We have a minimum resolution criteria in place for favicons. Favicons for Researchgate don't satisfy it and Glassdoor don't have a favicon with correct mime type. > 2. For politico the issue seems to be fixed and the icon is correctly displayed. Great. > 3. I have found another result that has a broken favicon: **xfinity**. @Abhishek could you please take a look over this? Looking into this one.