Open Bug 606170 Opened 14 years ago Updated 2 years ago

<input type='email' multiple> should accept a list of email ending with ","

Categories

(Core :: DOM: Forms, defect, P5)

defect

Tracking

()

People

(Reporter: mounir, Unassigned)

References

(Blocks 2 open bugs)

Details

(Keywords: dev-doc-needed, html5, Whiteboard: not-ready)

Attachments

(1 file)

Attached patch Patch v1Splinter Review
<input type='email' multiple>'s value has to be a set of comma-separated tokens [1] and all tokens have to be valid email addresses. This make "foo@foo.com, " not valid because the empty string isn't a valid email address.

The problem is, as long as "," isn't typed, the user is still editing the current email address. So, if we want to have a UI to select addresses, the UI will have to add "email@domain.com, " so the user can select multiple addresses easily. However, this will make the field invalid because of the ending comma.

Not that with this patch "foo@foo.com, " is valid but not " , " because even if it's ending with a comma, the first email address is not valid (the empty string).

[1] http://www.whatwg.org/specs/web-apps/current-work/multipage/common-microsyntaxes.html#set-of-comma-separated-tokens
Attachment #485028 - Flags: review?(jonas)
Attachment #485028 - Flags: approval2.0?
Blocks: 601209
Attachment #485028 - Flags: approval2.0? → approval2.0+
Whiteboard: [needs review]
Whiteboard: [needs review]
I will push that if bug 601209 can be fixed for Gecko 2.0. No need to push this patch otherwise.
This will be done after Gecko 2.0 given that bug 601209 will not be done for Gecko 2.0.
Attachment #485028 - Flags: approval2.0+ → approval2.0-
Whiteboard: not-ready
Priority: -- → P5
Blocks: html-forms
Component: DOM: Core & HTML → DOM: Forms

The bug assignee didn't login in Bugzilla in the last 7 months.
:hsinyi, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: mounir → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(htsai)
Flags: needinfo?(htsai)
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: