Closed Bug 1539699 Opened 2 years ago Closed 2 years ago

[about:addons] identify recommended extensions in about:addons

Categories

(Toolkit :: Add-ons Manager, enhancement, P2)

enhancement

Tracking

()

VERIFIED FIXED
mozilla69
Tracking Status
firefox68 --- fixed
firefox69 --- verified

People

(Reporter: ddurst, Assigned: mstriemer)

References

Details

(Whiteboard: [feature-scope])

Attachments

(1 file)

We want to be able to show the user which extensions they've installed that are recommended. This would be in the detail and list views.

This is intended to go along with discopane changes, but filed separately because it may be easier to deal with this when dealing with the rest of the HTML changes.

Depends on: 1540253
Priority: -- → P2

Release Note Request (optional, but appreciated)
[Why is this notable]: Users will see a visual indication on about:addons denoting which extensions they've installed that are recommended.
[Affects Firefox for Android]: n/a
[Suggested wording]: Find high-quality, secure extensions via the Recommended Extensions program. Look for these vetted extensions in about:addons.
[Links (documentation, blog post, etc)]: TBD

relnote-firefox: --- → ?

Looks like this and bug 1540173 have the same release notes request. Any preference which bug to associate it with?

Flags: needinfo?(ddurst)

This is because the Suggested wording is the same for both. Atsay -- this feature actually applies across a few things (this, 1540173, etc). But since there's no one place, this could be about discopane or it could be about the about:addons change.

Flags: needinfo?(ddurst) → needinfo?(atsay)

The relnotes apply to both, but if we need to choose one let's associate it with 1540173. Thanks!

Flags: needinfo?(atsay)

Thanks, I'll use that.

relnote-firefox: ? → ---
Depends on: 1554317
Assignee: nobody → rob
Status: NEW → ASSIGNED
Assignee: rob → mstriemer
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/02ee2cdf02b3
Show a recommended badge for recommended add-ons (HTML) r=mixedpuppy

Comment on attachment 9071733 [details]
Bug 1539699 - Show a recommended badge for recommended add-ons (HTML)

Beta/Release Uplift Approval Request

  • User impact if declined: Recommended add-ons installed from AMO will not be shown as recommended in Firefox.
  • Is this code covered by automated tests?: Yes
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce:
  • List of other uplifts needed: Bug 1540253
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Along with bug 1540253: these are fairly simple and small changes with test coverage.
  • String changes made/needed: No
Attachment #9071733 - Flags: approval-mozilla-beta?
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69

Rob asked to hold off on this uplift for now. lmk when this is good to go.

Flags: needinfo?(rob)

I fixes the (innocent) regressions, and it is ready to uplift:

  1. bug 1540253 (already mentioned in uplift request)
  2. bug 1539699 (this bug)
  3. bug 1559112 (remove warning message; verified by running the new test and confirming that there is no error message)
  4. bug 1559131 (skip test when MOZ_REQUIRE_SIGNING=1)

The last two patches are new.

Flags: needinfo?(rob)

Mark posted an updated uplift request with all bugs that I mentioned in comment 11 in https://bugzilla.mozilla.org/show_bug.cgi?id=1540253#c8

Comment on attachment 9071733 [details]
Bug 1539699 - Show a recommended badge for recommended add-ons (HTML)

approved for 68.0b11

Attachment #9071733 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
Depends on: 1559895

This issue is Verified as Fixed on Firefox 69.0a1 (20190617214029) under Windows 10 Pro 64-bit and MacOS High Sierra 10.13.6.

To verify the validity of the fix, the following prefs have been modified as follows:

Furthermore, for test purposes, several add-ons from dev have been marked with the recommended tag and pushed to the list of recommended addons displayed on about:addons.

When the above conditions are met, the ‘Recommended’ badge and the ‘Recommended’ string upon hovering over the badge are displayed.

However, while validating the fix, one potential issue has been encountered. We observed that the badge and string for recommended themes which are installed, are not displayed. For further details, please refer to: https://bugzilla.mozilla.org/show_bug.cgi?id=1559895.

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