Closed Bug 1387870 Opened 2 years ago Closed 2 years ago

Update Add-ons icon links after bug 1387731 landing

Categories

(Thunderbird :: Theme, enhancement)

enhancement
Not set

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 57.0

People

(Reporter: Paenglab, Assigned: Paenglab)

Details

Attachments

(1 file)

Bug 1387731 removed extensionGeneric-16.png. We don't need now to override this icon anymore.
Attached patch addonLink.patchSplinter Review
Remove the override and use the icon in AppMenu. The only visual difference is the now monochrome icon in AppMenu.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8894255 - Flags: review?(jorgk)
I can review this when I build next time.
(In reply to Richard Marti (:Paenglab) from comment #1)
> The only visual difference is the now monochrome icon in AppMenu.
Not only the AppMenu (Hamburger menu), also in the normal tab when opening the add-ons manager. Is that intentional?
Yes, this comes from the M-C patch.
Comment on attachment 8894255 [details] [diff] [review]
addonLink.patch

OK then.
Attachment #8894255 - Flags: review?(jorgk) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/b9c8cb277fc4
Use add-ons icon from bug 1387731. r=jorgk
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 57.0
You need to log in before you can comment on or make changes to this bug.