Closed
Bug 32376
Opened 25 years ago
Closed 25 years ago
Populting a SELECT list by setting its length
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
VERIFIED
FIXED
M16
People
(Reporter: omerk, Assigned: jst)
References
()
Details
Attachments
(1 file)
(deleted),
patch
|
Details | Diff | Splinter Review |
Cannot populate a SELECT list from another frame. Please see demo.
Reporter | ||
Updated•25 years ago
|
Comment 1•25 years ago
|
||
Confirmed that this works in NS 4.7. -> DOM0
Assignee: rogerl → jst
Component: Javascript Engine → DOM Level 0
QA Contact: rginda → desale
Updated•25 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
Reporter | ||
Comment 2•25 years ago
|
||
Some clarification: This bug occurs regardless of whether the SELECT list is in
a different frame.
Assignee | ||
Comment 3•25 years ago
|
||
Fixing this would be against the DOM Level 2 spec, the "lenght" property on
a select element (HTMLSelectElement) is readonly so you should not try to set
that property. I think, since "length" is currently not readonly in the
HTMLSelectElement definition in mozilla, that the decition to be backwards
compatible was made some time ago. Thus this is a valid bug that needs fixing.
rods, any comments about this or should I just go ahead and fix this in
nsHTMLSelectElement::SetLength()?
Status: NEW → ASSIGNED
OS: Windows NT → All
Hardware: PC → All
Reporter | ||
Comment 5•25 years ago
|
||
Any idea when this is going to be addressed?
Assignee | ||
Comment 7•25 years ago
|
||
Updating summbary to reflect the real problem.
Summary: Cross-frame SELECT list population → Populting a SELECT list by setting its length
Assignee | ||
Comment 9•25 years ago
|
||
Assignee | ||
Comment 10•25 years ago
|
||
rods, could you have a look at the patch I just attached?
Assignee | ||
Comment 11•25 years ago
|
||
This should be fixed now, the testcase works as expected now. Please note that
setting the length of a select is against the DOM tho...
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•