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

ASSIGNED
Assigned to

Status

()

Core
DOM: Core & HTML
ASSIGNED
7 years ago
2 months ago

People

(Reporter: mounir, Assigned: mounir)

Tracking

(Blocks: 2 bugs, {dev-doc-needed, html5})

Trunk
dev-doc-needed, html5
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: not-ready)

Attachments

(1 attachment)

4.30 KB, patch
sicking
: review+
Joe Drew (not getting mail)
: approval2.0-
Details | Diff | Splinter Review
(Assignee)

Description

7 years ago
Created attachment 485028 [details] [diff] [review]
Patch v1

<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?
(Assignee)

Updated

7 years ago
Blocks: 601209

Updated

7 years ago
Attachment #485028 - Flags: approval2.0? → approval2.0+
(Assignee)

Updated

7 years ago
Whiteboard: [needs review]
Attachment #485028 - Flags: review?(jonas) → review+
(Assignee)

Updated

7 years ago
Whiteboard: [needs review]
(Assignee)

Comment 1

7 years ago
I will push that if bug 601209 can be fixed for Gecko 2.0. No need to push this patch otherwise.
(Assignee)

Comment 2

7 years ago
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-
(Assignee)

Updated

7 years ago
Whiteboard: not-ready
You need to log in before you can comment on or make changes to this bug.