360926 - Identity editor needs a confirmation dialog on 'Delete'

Status: RESOLVED FIXED

(MailNews Core :: Account Manager, enhancement)

Description

[Dave Belfer-Shevett]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.7) Gecko/20060830 Firefox/1.5.0.7 (Debian-1.5.dfsg+1.5.0.7-2)
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.0.7) Gecko/20060830 Firefox/1.5.0.7 (Debian-1.5.dfsg+1.5.0.7-2)

When editing Identities, it is ridiculously simple to mistakenly delete an identity you've already setup.  Without a 'Cancel' button, this is an irreversible one-click function, with somewhat dire circumstances.

Either the Identities editor needs a 'cancel' function to undo changes, or the Delete button needs an 'Are you sure?' dialog.

Reproducible: Always

Steps to Reproduce:
1.Go to Accounts->Manage Identities
2.Spend 10 minutes setting up a new identity
3.Instead of hittin [ok], the mouse gets jostled and you hit Delete instead.
4.Curse loudly.

Actual Results:  
Identity is gone with no recourse.

Expected Results:  
Prompted with "Are you sure you want to delete this identity?"

Comment 1

[Magnus Melin [:mkmelin] (away, back Aug 4)]

Confirming RFE.

Comment 2

[:aceman]

Bwinton, are you OK with this?

Mistakenly deleting and then recreating the identity may assign it a different id so existing messages referencing the old ID may now have a problem (like Drafts or Unsent). But for the user the identity will look the same (same address and other values) so he may not understand it.

Comment 3

[:aceman]

Attachment: patch

Comment 4

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 638023 [details] [diff] [review]
patch

I think it's close, but not quite there.

As you can see in http://dl.dropbox.com/u/2301433/Screenshots/DeleteWhat.png there's no indication of which identity was selected (and indeed, it wasn't the "[e-mail]" identity, so the dialog was both incomplete and misleading ;).  I suggest something like:
+-------------------------------------------------+
| Deleting identity for [e-mail] |
|                                                 |
| Are you sure you want to delete the identity    |
| [e-mail]?                                 |
|                                [Cancel]   [OK]  |
+-------------------------------------------------+

In general, people don't read dialogs, and so it would be way better if we could support an "Undo" here instead, but since I suspect that's much more work, I think going this way is the best we can do in the short term.

So, ui-r-, so that I can see the new dialog wording, but other than that, it seems okay.

Thanks,
Blake.

Comment 5

[Jim Porter (:squib)]

(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #4)
> +-------------------------------------------------+
> | Deleting identity for [e-mail] |
> |                                                 |
> | Are you sure you want to delete the identity    |
> | [e-mail]?                                 |
> |                                [Cancel]   [OK]  |
> +-------------------------------------------------+

Let's use "Delete" instead of "OK" here. :)

Comment 6

[:aceman]

Attachment: patch v2

Yes, undo would need more thought. Maybe in a new bug if somebody requests it.

I fixed the strings and button as you both requested.

Comment 7

[Blake Winton (:bwinton) (:☕️)]

Comment on attachment 643592 [details] [diff] [review]
patch v2

Looks good.  ui-r=me!

Thanks,
Blake.

Comment 8

[Jim Porter (:squib)]

Comment on attachment 643592 [details] [diff] [review]
patch v2

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

Looks good. r=me

Comment 9

[Jim Porter (:squib)]

Checked in: http://hg.mozilla.org/comm-central/rev/42234d57ebdc

Comment 10

[Dave Belfer-Shevett]

As the originator of this bug almost 6 years ago, I'm ecstatic to see it resolved and committed.  Thank you everyone who worked on it!

Comment 11

[:aceman]

Hi, thanks for your reply!
When you try a version of TB (17 and up) that has the bug fixed, please change the status to VERIFIED. Thanks.