1473702 - Add "Learn More" to toolbar right-click context menus

Status: NEW

(WebExtensions :: Frontend, enhancement, P3)

Description

[Mike Conca [:mconca]]

Similar to bug 1457474, Firefox should natively add "Learn More" to the right-click context menu for both browser action buttons and page action buttons.  The "Learn More" menu item would take the user (in a new tab, presumably) to the extension's detail page on AMO.

UX has already mocked this up:
https://mozilla.invisionapp.com/share/T7DZLW5CR#/258734408_Right-Click-Extension

Comment 1

[Andrew Swan [:aswan]]

Not sure if I'm mis-reading the mockup but it looks like it actually goes to the detail page in about:addons (not AMO)

Comment 2

[Mike Conca [:mconca]]

I was going by Emanuela's comment in bug 14577474:
https://bugzilla.mozilla.org/show_bug.cgi?id=1457474#c7
which states it should go to the AMO detail page.

It looks like "Manage" already goes to the detail page in about:addons.  According to the mock, it should have gone to just about:addons?

Emanuela, would you mind providing some clarity?
"Manage" : goes to about:addons, OR the extension's detail page in about:addons?
"Learn More" : goes to extension's detail page in about:addons, OR the detail page on AMO?

Comment 3

[Andrew Swan [:aswan]]

If it's going to go to AMO, we need a specification for what should happen if the extension is unlisted.

Comment 4

[emanuela [ux]]

Attachment: Learn More From Contexual [e-mail]

Hey everyone, 

I attached how I'm expecting the logic to work when a user clicks on 'Learn More.' 

If this is too technically challenging, I'm open to discussing together alternatives!

Comment 5

[Andrew Swan [:aswan]]

This doesn't seem right, extensions can use extension pages for whatever they want, there's no way to tell if a particular extension page is an appropriate target for the "Learn More" action.

Also, I don't believe we currently store a reference to the AMO page for listed extensions...
It is available in the metadata update, but we currently don't keep it.  Adding it would be a small amount of extra work.

Comment 6

[Oriol Brufau [:Oriol]]

I think adding this at the top of the list is a bad idea.
The entries at the top should be the ones provided by the extension, because presumably these will be the ones that the user will use most of the time. I don't think users will frequently use "Learn more".
Also, currently the separation is clear: native entries at the bottom, extension ones at the top. Mixing them like in the invisionapp will confuse users.