Update icons on about:newtab
Categories
(Firefox :: New Tab Page, enhancement, P1)
Tracking
()
People
(Reporter: sfoster, Assigned: bigiri)
References
(Blocks 1 open bug)
Details
(Whiteboard: [proton-icons] [proton-uplift])
Attachments
(1 file)
48 bytes,
text/x-phabricator-request
|
pascalc
:
approval-mozilla-beta+
|
Details | Review |
We have updated icons for use on the new tab page.
Please see the procedure doc which has details on where to find the new icons, how to put together the patch and verify the result.
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Reporter | ||
Updated•3 years ago
|
Updated•3 years ago
|
Assignee | ||
Comment 1•3 years ago
|
||
Updated icons, skipped delete.svg, delete-20.svg, and edit.svg as I could not find these icons on Google Drive.
Updated•3 years ago
|
Updated•3 years ago
|
Pushed by bigiri@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/87a78e0df6cb Update icons on about:newtab r=sfoster,Mardak
Assignee | ||
Comment 3•3 years ago
|
||
Comment on attachment 9219008 [details]
Bug 1703027 - Update icons on about:newtab r=sfoster,mardak
Beta/Release Uplift Approval Request
- User impact if declined: Required for MR1 / Proton
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: Check that the icons in new tab, in Top Sites, and the UI accessible from the "..." menu when hovering over Top Sites has been updated to the new Proton icons.
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Icon changes only.
- String changes made/needed:
Assignee | ||
Updated•3 years ago
|
Assignee | ||
Comment 4•3 years ago
|
||
These icons can be found in the newtab page in the following locations:
- In Top Sites a search icon should be overlaid on the search provider icons.
- In Top Sites when you hover over an icon the menu should provide edit and delete options. In edit the second input field has an icon. In delete there should be an icon in the modal dialog.
Comment 5•3 years ago
|
||
bugherder |
Updated•3 years ago
|
Comment 6•3 years ago
|
||
Comment on attachment 9219008 [details]
Bug 1703027 - Update icons on about:newtab r=sfoster,mardak
Approved for 89 beta 7, thanks.
Comment 7•3 years ago
|
||
bugherder uplift |
Updated•3 years ago
|
Comment 8•3 years ago
|
||
The icons for the context menu for different items (top site tiles, pocket cards) are removed per bug 1681482. This has been verified using Fx89.0b11 and latest Fx90.0a1 on Windows 10 and Ubuntu 18.04.
Description
•