Closed Bug 1706935 Opened 8 months ago Closed 7 months ago

Update about:addons and about:preferences sidebar icons

Categories

(Firefox :: Theme, defect, P2)

Desktop
All
defect

Tracking

()

VERIFIED FIXED
90 Branch
Tracking Status
firefox89 --- verified
firefox90 --- verified

People

(Reporter: cbaica, Assigned: mconley)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [proton-aboutpages] [proton-icons] [priority:2a] [proton-uplift])

Attachments

(3 files)

Attached image extension icon bug

Affected versions

  • Fx89.0b2
  • Fx90.0a1

Affected platforms

  • Windows 10
  • Ubuntu 18.04
  • macOS 11

Steps to reproduce

  1. Launch Firefox.
  2. Navigate to about:addons
  3. Inspect all the available icons.

Expected result

  • All the icons are in accordance to the new proton design.

Actual result

  • The extension icon is still from the old design

Regression range

  • This is not a regression.

Additional notes

  • Other icons have the new design implemented (e.g. Plugins icon)
Blocks: 1694394
Whiteboard: [proton-aboutpages]

The theme icon was also reported in bug 1704746. Since the plugin icon has been updated and we do have new icons for most of these (I'm not seeing one for Dictionaries or Available Updates, but maybe we can find placeholders) it probably makes sense to update these icons. Alternatively we could revert the plugin icon to the old version for the Plugins category.

Romain, how should we proceed here? Currently the icons are mixing old and new. You said bug 1704746 was out of scope, but in my opinion we need to do something here to be consistent.

Flags: needinfo?(rtestard)
Attached image image.png

Likewise for about:preferences, it is mixing new and old icons

Has STR: --- → yes

Agreed that it would be nice to make this change for consistency reasons. Making this one a P2a. Do you think about:preferences and about :addons should be addressed in the same bug? If so about:preferences should be done with higher priority given it's a more visible area.

Flags: needinfo?(rtestard)
Priority: -- → P2
Whiteboard: [proton-aboutpages] → [proton-aboutpages] [priority:2a]

There are new icons for Comment 2 screenshot about:preferences
General = settings-20.svg (gear)
Home = home-20.svg (house)
Search = search-20.svg (magnifying glass)
Privacy & Security = lock-20.svg (lock)
Sync = sync-20.svg (2 arrows in circle)
Experiments = experiments-20.svg (atom + dots)

For about:add-ons
Recommendations = highlights-20.svg (3 sparkle stars)
Extensions = extension-20.svg (outline puzzle piece
Themes = themes-20.svg (wide paint brush)
Plugins = plugin-20.svg (lego block)

Whiteboard: [proton-aboutpages] [priority:2a] → [proton-aboutpages] [proton-icons] [priority:2a]
Assignee: nobody → mconley
Summary: The extension icon in about:addons is still from the old design → Update about:addons and about:preferences sidebar icons

about:addons

  • toolkit/themes/shared/extensions/category-themes.svg (themes.svg)
  • toolkit/themes/shared/extensions/category-discover.svg (highlights-20.svg)
  • toolkit/themes/shared/extensions/category-extensions.svg (extension-fill.svg)

about:preferences

  • browser/themes/shared/preferences/category-privacy-security.svg (lock-20.svg)
  • browser/themes/shared/preferences/category-general.svg (settings-20.svg)
  • browser/themes/shared/preferences/category-sync.svg (sync-20.svg)
  • browser/themes/shared/preferences/category-experiments.svg (experiments-20.svg)
  • browser/themes/shared/preferences/category-search.svg (search-20.svg)
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/dd857d5a6fb0
Update about:addons and about:preferences pane selector icons. r=preferences-reviewers,jaws
Blocks: proton-icons
Flags: needinfo?(mconley)
Pushed by mconley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d20682560678
Update about:addons and about:preferences pane selector icons. r=preferences-reviewers,jaws
Status: NEW → RESOLVED
Closed: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → 90 Branch

Comment on attachment 9218729 [details]
Bug 1706935 - Update about:addons and about:preferences pane selector icons. r?sfoster!

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?: 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): Asset swap for new Proton icons.
  • String changes made/needed: None.
Attachment #9218729 - Flags: approval-mozilla-beta?
Whiteboard: [proton-aboutpages] [proton-icons] [priority:2a] → [proton-aboutpages] [proton-icons] [priority:2a] [proton-uplift]
Attachment #9218729 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Hello,

Confirming this issue as verified fixed on 90.a1(20210519083222) and 89.0b14(20210518190425) on Ubuntu 20, Win10x64 and macOS11.

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.