The favicon is not correctly displayed in AS top - sites and highlights

VERIFIED FIXED

Status

()

P1
normal
VERIFIED FIXED
2 years ago
2 years ago

People

(Reporter: SimonB, Assigned: farhan)

Tracking

unspecified
All
iOS

Firefox Tracking Flags

(fxios-v7.0 affected, fxios7.0+)

Details

(Whiteboard: [mobileAS])

Attachments

(3 attachments)

(Reporter)

Description

2 years ago
Created attachment 8819295 [details]
IMG_0029.PNG

Build: 6dae35c6
Device: iPhone SE
iOS: 9.3.2

Prerequisites: 
- FxA with synced bookmarks and history 

Steps to reproduce:
1. Sign in to FxA (Sync)
2. Go to AS Top Sites - highlights

Actual results:
- The background color of several favicons is incorrect.
(Reporter)

Comment 1

2 years ago
Created attachment 8819296 [details]
IMG_0032.PNG

Updated

2 years ago
Whiteboard: [mobileAS]
(Assignee)

Updated

2 years ago
Assignee: nobody → fpatel
(Assignee)

Comment 2

2 years ago
Created attachment 8819483 [details]
Pull Request
Attachment #8819483 - Flags: review?(sleroux)
Status: NEW → ASSIGNED
Iteration: --- → 1.11
tracking-fxios: ? → 7.0+
Priority: -- → P1
(Assignee)

Comment 3

2 years ago
master https://github.com/mozilla-mobile/firefox-ios/commit/7b8e9767568b67768c32963114b855cffd31ac09
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Whiteboard: [mobileAS] → [mobileAS][needsuplift]
Attachment #8819483 - Flags: review?(sleroux)
v6.x 2e7d6dc06d1770cc0f40e1c0803f382262410665
Whiteboard: [mobileAS][needsuplift] → [mobileAS]
(Reporter)

Comment 5

2 years ago
Verifying as fix on master 51abcc7ca.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.