Closed Bug 1484152 Opened 3 years ago Closed 3 years ago

Add learn more link to autoplay doorhanger

Categories

(Firefox :: Site Identity, enhancement, P1)

enhancement

Tracking

()

VERIFIED FIXED
Firefox 63
Tracking Status
firefox63 --- verified

People

(Reporter: daleharvey, Assigned: daleharvey)

References

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1460208 +++

I didnt want to block https://bugzilla.mozilla.org/show_bug.cgi?id=1458249 on this since I dont know where that link is going to go yet
Component: Preferences → Site Identity and Permission Panels
Assignee: nobody → dharvey
https://bugzilla.mozilla.org/show_bug.cgi?id=1460208 adds it to the preferences, in the spec there is also a link in the doorhanger https://mozilla.invisionapp.com/share/N2MD6ZV8CMG#/screens/306583374
Comment on attachment 9001913 [details] [diff] [review]
0001-Bug-1484152-Add-learn-more-link-to-autoplay-doorhang.patch

Review of attachment 9001913 [details] [diff] [review]:
-----------------------------------------------------------------

::: browser/modules/PermissionUI.jsm
@@ +798,5 @@
>    },
>  
>    get popupOptions() {
> +    let learnMoreURL =
> +        Services.urlFormatter.formatURLPref("app.support.baseURL") + "block-autoplay";

nit: 2 spaces please
Attachment #9001913 - Flags: review?(jhofmann) → review+
Pushed by dharvey@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/80bb488144c9
Add learn more link to autoplay doorhanger. r=johannh
https://hg.mozilla.org/mozilla-central/rev/80bb488144c9
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 63
Verified, that "Learn more..." link is added to autoplay doorhanger on Nightly 63.0a1(20180828220157).
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.