Reorder "Manage Extension" and "Remove Extension" in browserAction context menu

VERIFIED FIXED in Firefox 64

Status

defect
VERIFIED FIXED
7 months ago
7 months ago

People

(Reporter: Oriol, Assigned: Oriol)

Tracking

64 Branch
mozilla65
Dependency tree / graph
Bug Flags:
qe-verify -

Firefox Tracking Flags

(firefox-esr60 unaffected, firefox63 unaffected, firefox64 fixed, firefox65 fixed)

Details

Attachments

(2 attachments)

Assignee

Description

7 months ago
Posted image screenshot.png
Bug 1401610 added a "Remove Extension" item in the toolbar icon context menu.

This was according to https://mozilla.invisionapp.com/share/T7DZLW5CR, where it looks good because there extra items provided by the add-on at the top of the context menu.

But if the add-on doesn't add any custom items there, then the first one is "Remove Extension". I don't think such a predominant position is appropriate for an option that most probably won't be used regularly.

And I think it's easier to click the first item accidentally. Some people don't read confirmation dialogs, so this may end up causing data loss for them. It will be their fault, but moving the item so that it's not that easy to click by mistake would be beneficial.

So I think it would be better to have "Manage Extension" first and "Remove Extension" second.

Emanuela (UX) approved the change in bug 1401610 comment 30.
Assignee

Updated

7 months ago
Keywords: checkin-needed

Comment 2

7 months ago
Pushed by nerli@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/777c4f39c2b8
Reorder "Manage Extension" and "Remove Extension" in browserAction context menu. r=Gijs
Keywords: checkin-needed

Comment 3

7 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/777c4f39c2b8
Status: ASSIGNED → RESOLVED
Last Resolved: 7 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Please request Beta approval on this when you get a chance.
Flags: qe-verify-
Flags: needinfo?(oriol-bugzilla)
Assignee

Comment 5

7 months ago
Verified as fixed on Firefox 65.0a1(3cff4bff1c9d) under Win 10 64-bit, Win 10 32-bit and Lubuntu 18.04.1 LTS 64-bit.
Status: RESOLVED → VERIFIED
Assignee

Comment 6

7 months ago
Comment on attachment 9021549 [details]
Bug 1503600 - Reorder "Manage Extension" and "Remove Extension" in browserAction context menu. r=Gijs

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1401610

User impact if declined: Worse UX, see comment 0.

Is this code covered by automated tests?: No

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: No

If yes, steps to reproduce: 

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Just reorders two context menu items

String changes made/needed:
Flags: needinfo?(oriol-bugzilla)
Attachment #9021549 - Flags: approval-mozilla-beta?
Comment on attachment 9021549 [details]
Bug 1503600 - Reorder "Manage Extension" and "Remove Extension" in browserAction context menu. r=Gijs

[Triage Comment]
Improved UX for the browserAction context menu. Approved for 64.0b6.
Attachment #9021549 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.