search

mozilla-mobile / firefox-ios

Firefox for iOS
Mozilla Public License 2.0
12.15k stars 2.91k forks source link

[Favicons refactor] - "Amazon.com"'s favicon doesn't load in share menu #22084

Open DianaABarladeanu opened 1 week ago

DianaABarladeanu commented 1 week ago

Steps to reproduce

  1. Install FF.
  2. Open share menu on amazon.com (either with hamburger menu or with context menu).
  3. Observe the favicon.

Expected behavior Tha favicon is instantly and correctly loaded.

Actual behavior The favicon doesn't load, a compass is displayed instead.

Device & build information

Notes Attachments:

https://github.com/user-attachments/assets/63cb8619-9887-4904-aed6-1d27fa652302

┆Issue is synchronized with this Jira Task

OrlaM commented 1 week ago

@ih-codes this one is a little different. The Amazon logo should exist in the cache at this point when the share sheet is open.

ih-codes commented 1 week ago

@ih-codes this one is a little different. The Amazon logo should exist in the cache at this point when the share sheet is open.

Agreed, this one needs some investigation. I think I saw a few tickets where this share sheet doesn't seem to load favicons.