Closed
Bug 1239028
Opened 10 years ago
Closed 10 years ago
Make Spotlight/NSUserActivity use a Favicon on first visit to a site.
Categories
(Firefox for iOS :: Favicons, defect)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | + | --- |
People
(Reporter: jhugman, Assigned: jhugman)
References
Details
Attachments
(1 file)
Favicons are fragile.
This is a regression in the FaviconManager that causes the Favicons to not be attached to an NSUserActivity when the site has not been visited before.
iOS uses a fallback of the appicon if no icon is available.
This is a followup to Bug 1221608.
Updated•10 years ago
|
Blocks: 1221608
tracking-fxios:
--- → ?
Updated•10 years ago
|
| Assignee | ||
Updated•10 years ago
|
Assignee: nobody → jhugman
Status: NEW → ASSIGNED
| Assignee | ||
Comment 1•10 years ago
|
||
Comments in the PR.
Corner case discussed there is fixable by Bug 1244262.
Attachment #8713784 -
Flags: review?(sleroux)
Comment 2•10 years ago
|
||
Comment on attachment 8713784 [details] [review]
Pull request
Code looks good and work for me in the simulator!
Attachment #8713784 -
Flags: review?(sleroux) → review+
| Assignee | ||
Comment 3•10 years ago
|
||
Merged!
| Assignee | ||
Updated•10 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•