Add weather icons for weather suggestions
Categories
(Firefox :: Address Bar, task, P1)
Tracking
()
People
(Reporter: mcheang, Assigned: mcheang)
References
Details
(Whiteboard: [snt])
Attachments
(1 file, 1 obsolete file)
(deleted),
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details |
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
This patch receives a weather icon id from our merino server. We then use that
icon id and map it to a specific weather icon svg file within
urlbar-dyanmic-result.css.
Assignee | ||
Comment 2•2 years ago
|
||
This patch receives a weather icon id from our merino server. We then use that
icon id and map it to a specific weather icon svg file within
urlbar-dyanmic-result.css.
Updated•2 years ago
|
Assignee | ||
Comment 3•2 years ago
|
||
Comment on attachment 9312285 [details]
Bug 1810250 - Add weather icons for weather suggestions in the urlbar.r=adw
Beta/Release Uplift Approval Request
- User impact if declined: We want to get the icons out to beta users for weather suggestions in the urlbar for foxfooding in beta. If this patch is declined, the user will see a star placeholder for the weather icon and not the actual weather icon, such as rainy, sunny, snowy, etc.
This code is not covered by automated test because it only contains changes for ui styling by adding svg files. There is already existing code for weather suggestions in place else where.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): This patch is not risky because it only invovles UI updates by adding the weather icon svg files. There's no potential for this patch to block the functioning of the urlbar.
- String changes made/needed: N/A
- Is Android affected?: No
Assignee | ||
Comment 4•2 years ago
|
||
[Tracking Requested - why for this release]:
Comment 6•2 years ago
|
||
Backed out for causing xpc failures in browser/components/urlbar/tests/quicksuggest/unit/test_weather.js
Backout link: https://hg.mozilla.org/integration/autoland/rev/1a781d5323e70acfa7ab152fb2a8cb9e554b1800
Comment 7•2 years ago
|
||
Comment on attachment 9312285 [details]
Bug 1810250 - Add weather icons for weather suggestions in the urlbar.r=adw
Not approving for beta as it was backed out on mozilla-central.
Updated•2 years ago
|
Assignee | ||
Comment 8•2 years ago
|
||
Thanks for backing it out. I've identified the failure and I am fixing it currently.
Assignee | ||
Comment 9•2 years ago
|
||
Okay I've updated my patch with the fix.
Comment 10•2 years ago
|
||
Assignee | ||
Comment 11•2 years ago
|
||
Comment on attachment 9312285 [details]
Bug 1810250 - Add weather icons for weather suggestions in the urlbar.r=adw
I fixed the test failure for test_weather.js
and pushed to TRY: https://treeherder.mozilla.org/jobs?repo=try&revision=e3d27beb84d439a1ae21740c1ad99b58dc692b8f&selectedTaskRun=aSFmhvY_RGqunDdlLS_p_Q.0
After investigating, the other tests seem unrelated to this patch.
Re-requesting beta uplift.
Comment 12•2 years ago
|
||
bugherder |
Comment 13•2 years ago
|
||
Comment on attachment 9312285 [details]
Bug 1810250 - Add weather icons for weather suggestions in the urlbar.r=adw
Approved for 110 beta 3, thanks
Comment 14•2 years ago
|
||
bugherder uplift |
Comment 15•2 years ago
|
||
I have verified this issue on Firefox Release 111.0.1 (Build ID: 20230321111920) on Firefox Beta 112.0b9 (Build ID: 20230330182947) and the latest Nightly 113.0a1 (Build ID: 20230402214746) on Windows 10 x64, macOS 12.4 and Linux Ubuntu 20.04 x64.
- An icon is displayed each time a weather result is triggered.
- The icon is specific to the described forecast in the weather result.
Description
•