Identity editor needs a confirmation dialog on 'Delete'

RESOLVED FIXED in Thunderbird 17.0


13 years ago
7 years ago


(Reporter: dbs, Assigned: aceman)


(Blocks 1 bug)

Thunderbird 17.0
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)



(1 attachment, 1 obsolete attachment)

5.57 KB, patch
: review+
: review+
: ui-review+
Details | Diff | Splinter Review
User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv: Gecko/20060830 Firefox/ (Debian-1.5.dfsg+
Build Identifier: Mozilla/5.0 (X11; U; Linux i686; en-US; rv: Gecko/20060830 Firefox/ (Debian-1.5.dfsg+

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?"
Confirming RFE.
Severity: normal → enhancement
Ever confirmed: true
OS: Windows XP → All
Hardware: PC → All
Assignee: mscott → nobody
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.
Assignee: nobody → acelists
Product: Thunderbird → MailNews Core
QA Contact: account-manager → account-manager
Version: unspecified → Trunk
Posted patch patch (obsolete) — Splinter Review
Attachment #638023 - Flags: ui-review?(bwinton)
Comment on attachment 638023 [details] [diff] [review]

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

As you can see in there's no indication of which identity was selected (and indeed, it wasn't the "" identity, so the dialog was both incomplete and misleading ;).  I suggest something like:
| Deleting identity for |
|                                                 |
| Are you sure you want to delete the identity    |
| test@test.test?                                 |
|                                [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.

Attachment #638023 - Flags: ui-review?(bwinton) → ui-review-
(In reply to Blake Winton (:bwinton - Thunderbird UX) from comment #4)
> +-------------------------------------------------+
> | Deleting identity for |
> |                                                 |
> | Are you sure you want to delete the identity    |
> | test@test.test?                                 |
> |                                [Cancel]   [OK]  |
> +-------------------------------------------------+

Let's use "Delete" instead of "OK" here. :)
Posted patch patch v2Splinter Review
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.
Attachment #638023 - Attachment is obsolete: true
Attachment #643592 - Flags: ui-review?(bwinton)
Attachment #643592 - Flags: review?(squibblyflabbetydoo)
Blocks: 314806
Comment on attachment 643592 [details] [diff] [review]
patch v2

Looks good.  ui-r=me!

Attachment #643592 - Flags: ui-review?(bwinton) → ui-review+
Comment on attachment 643592 [details] [diff] [review]
patch v2

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

Looks good. r=me
Attachment #643592 - Flags: review?(squibblyflabbetydoo) → review+
Attachment #643592 - Flags: review?(iann_bugzilla)
Attachment #643592 - Flags: review?(iann_bugzilla) → review+
Keywords: checkin-needed
Checked in:
Closed: 7 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 17.0
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!
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.
Blocks: 792334
You need to log in before you can comment on or make changes to this bug.