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)

defect

Tracking

()

People

(Reporter: mounir, Unassigned)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-needed, html5)

Attachments

(1 file, 1 obsolete file)

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
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?
Assignee: nobody → andrew.quartey
Attached patch Impl (obsolete) (deleted) — Splinter Review
Attachment #795013 - Flags: review?(mounir)
Attached patch Impl (deleted) — Splinter Review
Wrong patch initially.
Attachment #795013 - Attachment is obsolete: true
Attachment #795013 - Flags: review?(mounir)
Attachment #795201 - Flags: review?(mounir)
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)
Priority: -- → P3
Old patch on bug. Heads up Overholt.
Flags: needinfo?(overholt)
I don't think the assignee is still actively working on this.
Assignee: andrew.quartey → nobody
Flags: needinfo?(overholt)
Priority: P3 → P5
Blocks: html-forms
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: