Closed Bug 1525092 Opened 5 years ago Closed 5 years ago

[about:addons] Confirm removing an extension before removing it (HTML)

Categories

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

enhancement

Tracking

()

VERIFIED FIXED
mozilla67
Tracking Status
firefox67 --- verified
firefox68 --- verified

People

(Reporter: mstriemer, Assigned: mstriemer)

References

Details

(Whiteboard: [feature-scope])

Attachments

(2 files)

When the remove button is clicked the user should be prompted to confirm their decision.

Example interaction: https://mozilla.invisionapp.com/share/W2DW6NYX9

Whiteboard: [feature-scope]

Emanuela, should we continue to offer undo support? If we do that do we need to confirm the removal?

We could potentially provide the option to undo in the "You've removed Add-on X" banner.

Flags: needinfo?(emanuela)

My preference is to have the undo button in the banner. That way the experience of removing an extension (with a modal confirmation) is the same from the card and the browser action button.

(In reply to Amy Tsay [:amyt] from comment #2)

My preference is to have the undo button in the banner. That way the experience of removing an extension (with a modal confirmation) is the same from the card and the browser action button.

Hey, Amy, how is the experience the same if we keep the undo button in the banner? Isn't it the opposite, by adding a feature the user can't access if they remove from the toolbar?

This is how I think the flow should go:

  • Someone shows the intention to get rid of an extension by clicking remove (from the toolbar or the extensions manager, doesn't matter)
  • Firefox displays a confirmation modal (so, in case someone clicked by mistake, they can decide to go back). In the confirmation modal, we can show the option to report the extension
  • The extension is removed (if the user is in the add-ons manager, we show a confirmation in a message bar component)

Anyway, if we want to keep the Undo feature in the extensions manager, my suggestion is to display it in the message bar. It will make sense to ask Meridel or Michelle if there is a betters string for it ("Undo" doesn't sound Photon).

Flags: needinfo?(emanuela)

I think we're saying the same thing, sort of? That a confirmation modal should show regardless of where you remove from, and to have an Undo button in the banner/message bar after removal in the add-on manager.

Ni'ing Meridel for confirmation on "Undo" wording.

Flags: needinfo?(mwalkington)

Looks like the Invision file has been archived:https://mozilla.invisionapp.com/share/W2DW6NYX9

Emanuela, can you please share with me so I can see what this looks like? Thank you.

Flags: needinfo?(mwalkington) → needinfo?(emanuela)

I have access now. I will review and comment back here on copy.

Flags: needinfo?(emanuela)

I don't think we need both a confirmation dialogue AND an undo option. When deciding between the two, I recommend going the confirmation route for the following reasons:

-Confirmation is best suited for "destructive" actions, like removing or deleting something (as the user is doing in this case)
-Undo is best for repetitive actions, where you don't want to be interrupted in your flow (like sending sending an email and not wanting to be asked to confirm the action every time)

Emanuela also pointed out that the confirmation dialogue allows us to include the option to report the extension.

Amy, if you still feel strongly about including an "Undo" function, we could try to work it into the message bar there, but it will be harder to replicate if the user removes from extension the toolbar.

Let us know your thoughts! Thanks.

Flags: needinfo?(atsay)

When is this being implemented? We are working on the test protocol for abuse reporting and this should reflect the direction we decide on (confirmation vs. undo).

Results from yesterday's meeting:

  • We aligned on the modal and banner changes that will apply to both the conversion to HTML and abuse reporting.
  • In both flows, there will be a modal after clicking Remove or Disable. When abuse reporting lands, there will be a checkbox on the modal that triggers the reporting flow.
  • In both cases, a message banner shows after the modal closes. "Undo" will be a final option in both.
Flags: needinfo?(atsay)
Blocks: 1533795
No longer blocks: 1505924
Priority: P2 → P1
Assignee: nobody → mstriemer
Pushed by mstriemer@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0fb1354119e7
Confirm removing an extension in HTML about:addons r=jaws
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla67

STR

  1. Enable the HTML about:addons views (extensions.htmlaboutaddons.enabled)
  2. Install an extension
  3. View the Extension list page
  4. Open the more options menu (... button)
  5. Select Remove

A confirmation dialog should be shown. Selecting Remove will uninstall the extension, Cancel will dismiss the dialog.

Attached image Bug1525092.gif

This issue is verified as fixed on Firefox 68.0a1 (20190405111221) and Firefox 67.0b8 - build2 (20190404130536)under Win 7 64-bit and Mac OS X 10.14.1.

Please see the attached video.

Status: RESOLVED → VERIFIED
Summary: Confirm removing an extension before removing it (HTML) → [about:addons] Confirm removing an extension before removing it (HTML)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: