search

mozilla-mobile / firefox-ios

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

[Favicons refactor][Sponsored Shortcuts] - `T-Mobile.com` favicon is blurry everywhere #22096

Open abodea opened 1 week ago

abodea commented 1 week ago

Steps to reproduce

  1. Install FF.
  2. Access the sponsored T-Mobile.com.
  3. Add it to shortcuts.
  4. Add it to bookmarks.
  5. Add website shortcuts widget.
  6. Observe the favicon in all the sections.

    Expected behavior

    The favicon is clear.

    Actual behavior

    T-Mobile.com sponsored favicon is blurry everywhere

    Device & build information

    • Device: iPhone 15 Pro (17.5).
    • Build version: v131 (45403)
    • First seen version: N/A

      Notes

      Attachments:

https://github.com/user-attachments/assets/3f1c626b-7b34-4479-871b-01d63431d83f

┆Issue is synchronized with this Jira Task

nbhasin2 commented 5 days ago

Blurry icons will be moving to S4 category and referencing the following PRs for it which improves it a bit on widget side https://github.com/mozilla-mobile/firefox-ios/pull/22056 - Widget Only https://github.com/mozilla-mobile/firefox-ios/pull/22101 - Widget + App