Closed Bug 1159181 Opened 9 years ago Closed 6 years ago

[Email/UI] Compose UI: Known invalid email address bubbles could be styled to indicate they are in error and/or space/comma/semicolon could not create a bubble when in error

Categories

(Firefox OS Graveyard :: Gaia::E-Mail, enhancement)

ARM
Gonk (Firefox OS)
enhancement
Not set
normal

Tracking

(tracking-b2g:+, b2g-v2.2 affected, b2g-master affected)

RESOLVED WONTFIX
tracking-b2g +
Tracking Status
b2g-v2.2 --- affected
b2g-master --- affected

People

(Reporter: wangxin, Unassigned)

Details

Attachments

(2 files)

[1.Description]:
[Nexus 5][v2.2&3.0][Email]:When you add e-mail address, if you've added an incorrct one, it will be marked red. But if you add a wrong e-mail address and the seperate them with a semicolon, the wrong e-mail address will not be marked red.
Found Time: 04:31
See video: 0431.MP4
See log:"logcat_0431.txt"

[2.Testing Steps]: 
Premise: Have an account loged in device
1. Launch e-mail app.
2. Tap the Compose on the top of right.
3. Input some incorrect e-mail addresses as recipients.
4. Use a semicolon to separate recipients

[3.Expected Result]: 
4.The incorrect e-mail address should be marked red.

[4.Actual Result]: 
4.The incorrect e-mail address isn't marked red.

[5.Reproduction build]: 
Flame 2.2 version(Affected):
Build ID               20150427162501
Gaia Revision          9f6b1b9082662ba2c14168fc66bb02b4df3141e5
Gaia Date              2015-04-27 20:41:33
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/e79c19bf19bf
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150427.195038
Firmware Date          Mon Apr 27 19:50:51 EDT 2015
Bootloader             L1TC000118D0

Device: Flame 3.0 (affected)
Build ID               20150427160201
Gaia Revision          0636405f0844bf32451a375b2d61a2b16fe33348
Gaia Date              2015-04-27 16:42:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/caf25344f73e
Gecko Version          40.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150427.192938
Firmware Date          Mon Apr 27 19:29:49 EDT 2015
Bootloader             L1TC000118D0


Device: Nexus5 2.2 (affected)
Build ID               20150427002504
Gaia Revision          265ca0bc9408c21fc4b25a259fcee7fb642cd06b
Gaia Date              2015-04-24 19:13:28
Gecko Revision         https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/1908685d798d
Gecko Version          37.0
Device Name            hammerhead
Firmware(Release)      5.1
Firmware(Incremental)  eng.cltbld.20150427.035747
Firmware Date          Mon Apr 27 03:58:04 EDT 2015
Bootloader             HHZ12f

Device: Nexus5 3.0 (affected)
Build ID               20150427160201
Gaia Revision          0636405f0844bf32451a375b2d61a2b16fe33348
Gaia Date              2015-04-27 16:42:28
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/caf25344f73e
Gecko Version          40.0a1
Device Name            hammerhead
Firmware(Release)      5.1
Firmware(Incremental)  eng.cltbld.20150427.192534
Firmware Date          Mon Apr 27 19:25:51 EDT 2015
Bootloader             HHZ12f

[6.Reproduction Frequency]: 
Always Recurrence,10/10

[7.TCID]: 
Free Test

[8. Note]:
Some users prefer using Space to separate recipients, but some other would prefer ; (semicolon). 
When user is sending an email to many recipients (ex.: 10), if the wrong emails can be separated by ; , it will be hard for you to find out which email is the incorrect one. This would be time-consuming.  In my opinion, the ; cannot be used to separate wrong email addresses, just like Space does.
Attached video Bug video: 0431.mp4
(In reply to SandKing from comment #0)
> [8. Note]:
> Some users prefer using Space to separate recipients, but some other would
> prefer ; (semicolon). 
> When user is sending an email to many recipients (ex.: 10), if the wrong
> emails can be separated by ; , it will be hard for you to find out which
> email is the incorrect one. This would be time-consuming.  In my opinion,
> the ; cannot be used to separate wrong email addresses, just like Space does.

So the difference here is that when we see a space character (' '), we will only generate a bubble if we've seen an '@' character, but for both comma (',') and semicolon (';') we always generate a bubble.  I agree the space behaviour is superior and probably would be appropriate for the command and semicolon cases too.  But if we're going to show an error on bubbles, we could perhaps just change things to make it so that we won't generate a bubble in any of those cases if we are confident that the address is invalid.
Severity: normal → enhancement
Summary: [Email]When you add a wrong e-mail address and then a semicolon between recipients ,the incorrect e-mail address is not marked red. → [Email/UI] Compose UI: Known invalid email address bubbles could be styled to indicate they are in error and/or space/comma/semicolon could not create a bubble when in error
Hi Juwei, Could you check this please, thanks.
Flags: needinfo?(jhuang)
I guess it would be easier to show bubble when we see "@" with either space, command or semicolon character.
Flags: needinfo?(jhuang)
Any backlog meta bug or list we should add this to?
NI Dylan for decision and assignment.
Flags: needinfo?(doliver)
tracking-b2g: --- → +
Flags: needinfo?(doliver)
Firefox OS is not being worked on
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: