Closed Bug 1167408 Opened 10 years ago Closed 10 years ago

Correct "globe icon" in places affected by bug 1162254

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED INVALID

People

(Reporter: antlam, Unassigned)

References

Details

From bug 1162254 (In reply to Anthony Lam (:antlam) from comment #15) > Ally, is it me or has this replaced the globe icon in more places than just > the about:passwords list? I'm looking at the "missing" favicon (globe) we > use for display lists of links in all our panels UI, and the globe in the > Top site tiles. There may be some I've missed. > > Also, the globe in the tabs tray looks bigger now as well, or is it just me?
NI-ing Ally and Chenxia here. We do need to fix this.
Flags: needinfo?(liuche)
Flags: needinfo?(ally)
(In reply to Anthony Lam (:antlam) from comment #0) > From bug 1162254 > > (In reply to Anthony Lam (:antlam) from comment #15) > > Ally, is it me or has this replaced the globe icon in more places than just > > the about:passwords list? I'm looking at the "missing" favicon (globe) we > > use for display lists of links in all our panels UI, and the globe in the > > Top site tiles. There may be some I've missed. > > > > Also, the globe in the tabs tray looks bigger now as well, or is it just me? Bug 1162254 just used the existing icon that we already had in the tree... nothing besides about:passwords should be any different. We can use this bug to update the icon everywhere, but I think we should continue to have a single icon for the sake of simplicity and APK size.
Agree, we should re-use the globe assets (uploaded updated ones in bug 1164715) where possible. These were updated to follow our color palette in our UI. We can use these in the tablet tabs tray, mobile tabs tray, as well as empty state in top site panel tiles, and our UI for our links in list view (bookmarks, history, synced tabs, etc). Can we scale down the same globe asset we're using in bug 1164715 in the list view?
Tried digging some more and can't find out what we originally saw so I'm going to close this one.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → INVALID
Flags: needinfo?(liuche)
Flags: needinfo?(ally)
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.