805335 - [email] account deletion: implement "delete account" button per UX spec

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect, P1)

Description

[Dylan Oliver [:doliver]]

The current method to remove an email account is not as specified in the UX wireframes and is difficult to discover. 

We have a v1 requirement to allow account deletion and feel that we still need to improve the user experience to satisfy it.

Refer to pages 6 & 9 of this spec: https://www.dropbox.com/sh/ygwfxk6chpshxdj/ct4nwRk2r3/Apps/Email/mail-settings.pdf

Comment 1

[Steve Chung [:steveck]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/6051

Comment 2

[Andrew Sutherland [:asuth] (he/him)]

https://github.com/mozilla-b2g/gaia/pull/6051
  landed on gaia/master:
https://github.com/mozilla-b2g/gaia/commit/3d15f744cc3d1194eb4a98473fbe96d10171d9e0

Comment 3

[Andrew Sutherland [:asuth] (he/him)]

Steve, I just realized the delete button did not add a confirmation prompt.  Please create a follow-up patch that confirms the user wants to delete the account with a prompt.  Since Casey is out, Josh can ideally provide the phrasing for that.

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

Er, the prompt is already defined on page 11 of the wireframes.  But note that the use of "loose" is a typo; in american english, that should be "lose".  I'm leaving Josh copied since I think the phrasing is a bit awkward/misleading:

"Are you sure you want to delete 
[e-mail] mail account?  
You will loose any downloaded mail"

No mail will really be lost since the mail will still be on the server. It might be too wordy, but "You will have to re-download your mail if you re-create the account." seems more accurate.

Comment 5

[Steve Chung [:steveck]]

Attachment: Patch for account delete confirm prompt

Comment 6

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 677688 [details]
Patch for account delete confirm prompt

r=asuth with the requested changes (new l10n id's) made.  Feel free to land once you do that.  Thanks!

Comment 7

[Steve Chung [:steveck]]

New l10n added , thanks for the comment.

Comment 8

[Josh Carpenter [:jcarpenter]]

Also worth nothing that we (UX + copy writer) will be reviewing all strings before string-freeze, so this will still get one more quality pass.

Comment 9

[Mila Davydova]

Verified with Unagi, build ID 20121231070201
User is able to delete an email account. After selecting "Delete account" button user is presented with a prompt message: "You will have to re-download your mail if you re-create the account."