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)
Core
DOM: Forms
Tracking
()
NEW
People
(Reporter: mounir, Unassigned)
References
(Blocks 2 open bugs)
Details
(Keywords: dev-doc-needed, html5, Whiteboard: not-ready)
Attachments
(1 file)
(deleted),
patch
|
sicking
:
review+
joe
:
approval2.0-
|
Details | Diff | Splinter 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?
Updated•14 years ago
|
Attachment #485028 -
Flags: approval2.0? → approval2.0+
Reporter | ||
Updated•14 years ago
|
Whiteboard: [needs review]
Attachment #485028 -
Flags: review?(jonas) → review+
Reporter | ||
Updated•14 years ago
|
Whiteboard: [needs review]
Reporter | ||
Comment 1•14 years ago
|
||
I will push that if bug 601209 can be fixed for Gecko 2.0. No need to push this patch otherwise.
Reporter | ||
Comment 2•14 years ago
|
||
This will be done after Gecko 2.0 given that bug 601209 will not be done for Gecko 2.0.
Updated•14 years ago
|
Attachment #485028 -
Flags: approval2.0+ → approval2.0-
Reporter | ||
Updated•14 years ago
|
Whiteboard: not-ready
Updated•6 years ago
|
Priority: -- → P5
Updated•6 years ago
|
Blocks: html-forms
Updated•3 years ago
|
Component: DOM: Core & HTML → DOM: Forms
Comment 3•3 years ago
|
||
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)
Updated•3 years ago
|
Flags: needinfo?(htsai)
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•