Closed
Bug 184925
Opened 22 years ago
Closed 22 years ago
isindex.form always == null
Categories
(Core :: DOM: Core & HTML, defect)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
FIXED
People
(Reporter: jst, Assigned: peterv)
References
()
Details
Attachments
(1 file)
|
864 bytes,
patch
|
sicking
:
review+
jst
:
superreview+
|
Details | Diff | Splinter Review |
The form property on isindex elements is always null, even if the isindex element is contained inside a form. This is a bug that shows up when running the W3C DOM TS, Level 2 HTML.
| Reporter | ||
Comment 1•22 years ago
|
||
Load the URL, and paste this into the URL bar:
javascript:alert(document.getElementsByTagName('isindex')[0].form);
You'll see nothing, when you should see "[Object HTMLFormElement]"..form on an isindex doesn't make sense and should be removed from the spec. <form>s and <isindex>s never interact in any way. The <isindex> is not submitted when a form is submitted, and the <isindex> does not take any information from a <form> tag to decide on how or where to submit its information. See http://www.w3.org/TR/REC-html32#isindex
Comment 3•22 years ago
|
||
100% agreed. I am told we don't want to modify the spec, though, and we want to be compliant with the tests for good PR. Johnny, Peter, with the knowledge that this may get into an errata (probably everyone agrees on it), perhaps we can simply get it removed from the *tests*?
hmmm.. that property is in DOM-HTML-L1 so i guess it's way too late to remove it now. IMHO it should always be null though since there is no form that the element "belongs" to.
| Assignee | ||
Comment 5•22 years ago
|
||
| Assignee | ||
Updated•22 years ago
|
Attachment #109750 -
Flags: superreview?(jst)
Attachment #109750 -
Flags: review?(bugmail)
Comment on attachment 109750 [details] [diff] [review] v1 strange as this whole thing is, r=me
Attachment #109750 -
Flags: review?(bugmail) → review+
| Reporter | ||
Comment 7•22 years ago
|
||
Comment on attachment 109750 [details] [diff] [review] v1 Nice! sr=jst
Attachment #109750 -
Flags: superreview?(jst) → superreview+
| Assignee | ||
Comment 8•22 years ago
|
||
This was checked in a while ago.
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
Component: DOM: HTML → DOM: Core & HTML
QA Contact: stummala → general
You need to log in
before you can comment on or make changes to this bug.
Description
•