1427622 - Extension install prompt hides permissions when there are too many

Status: NEW

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

Description

[seanlinsley]

Attachment: Installing Better TweetDeck

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:59.0) Gecko/20100101 Firefox/59.0
Build ID: 20171227220103

Steps to reproduce:

Try to install Better TweetDeck: https://addons.mozilla.org/en-US/firefox/addon/better-tweetdeck-17


Actual results:

The extension requests so many permissions that they're summarized down to "access your data on 15 other sites".


Expected results:

Instead, the UI should show all permissions hidden behind a scrollbar.

Another option would be to update addons.mozilla.org so that it lists the permissions requested before you install the extension.

Comment 1

[seanlinsley]

Hmm, the browser version selection UI when submitting this bug only had "trunk" as an option to represent Firefox Nightly. Seems like that could be worded better.

Comment 2

[seanlinsley]

I originally posted this issue on Twitter: https://twitter.com/seanlinsley/status/948236715432513536

Comment 3

[Andrew Swan [:aswan]]

This was a deliberate choice when the notifications were first implemented.
Markus, any thoughts on how we could expose the complete list?

Comment 4

[Markus Jaritz [:markus] (UX)]

Emanuela was working on this. She proposed to grow the panel. Forwarding the needinfo to her.
I think simplify permissions for the case of may very similar permissions might make sense.

Comment 5

[Markus Jaritz [:markus] (UX)]

here is a link to the design doc where we talked about that: https://www.figma.com/file/HrLiKUwoLQZsIUIVBKM8Wnnu/Install-Flow-showing-Permissions?node-id=898%3A395

Comment 6

[emanuela [ux]]

This bug feels like a duplicate of 1387695.

Anyway.

I think we should grow the permission door-hanger to be as long as the menu (currently is ~500px).
However, they're going to be some situation we're going to group similar permissions - like in this one.

The solution consists in:
- Group together all the related domains 
- Show the full list of domains in a second level menu

(here a low-fidelity mock from the other bug https://bug1387695.bmoattachments.org/attachment.cgi?id=8901766)