Open
Bug 623538
Opened 14 years ago
Updated 2 years ago
Update how <input type='email' multiple> to follow what the specifications currently say
Categories
(Core :: DOM: Core & HTML, defect, P5)
Core
DOM: Core & HTML
Tracking
()
NEW
People
(Reporter: mounir, Unassigned)
References
(Blocks 1 open bug)
Details
(Keywords: dev-doc-needed, html5)
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Details | Diff | Splinter Review |
The specifications about <input type='email' multiple> have just changed. We should update our implementation.
See: http://html5.org/tools/web-apps-tracker?from=5739&to=5740
Comment 1•12 years ago
|
||
This looks challenging enough.I have already read up the current specs from http://www.whatwg.org/specs/web-apps/current-work/multipage/common-input-element-attributes.html#attr-input-multiple and the little pop-up widget to the side indicates firefox has partial support for the multiple attribute. Mounir, is this dependent on another feature awaiting implementation?
Updated•12 years ago
|
Assignee: nobody → andrew.quartey
Comment 2•11 years ago
|
||
Attachment #795013 -
Flags: review?(mounir)
Comment 3•11 years ago
|
||
Wrong patch initially.
Attachment #795013 -
Attachment is obsolete: true
Attachment #795013 -
Flags: review?(mounir)
Attachment #795201 -
Flags: review?(mounir)
Reporter | ||
Comment 4•11 years ago
|
||
Comment on attachment 795201 [details] [diff] [review]
Impl
Review of attachment 795201 [details] [diff] [review]:
-----------------------------------------------------------------
Currently, Gecko only sanitize values that are set via script or when the element is parsed. We do not sanitize the value set by the user assuming that it will not parse constraint validation. I also do not see where in the specifications it is mentioned that the value sanitization algorithm should be run for user-entered values. A consequence of that is with this code, a valid value would not pass sanitization and the win in sanitizing is to make sure the servers can have simple values to process.
Before taking that patch, I think we should figure out if there is a problem in our implementation of value sanitization or in the HTML specifications.
Attachment #795201 -
Flags: review?(mounir)
Updated•6 years ago
|
Priority: -- → P3
Comment 6•6 years ago
|
||
I don't think the assignee is still actively working on this.
Assignee: andrew.quartey → nobody
Flags: needinfo?(overholt)
Priority: P3 → P5
Updated•6 years ago
|
Blocks: html-forms
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•