Reassigning to owner of DOM Level 0 (sorry, vidur.)
20 years ago
Summary: Form selection field selectedIndex not working properly → [4.xP]HTMLSelectElement.options.selectedIndex not supported
I would expect "document.outputdebug.foobie.selectedIndex" to work, but not with ".options" in the middle. For some reason, though, this does work in 4.x. How annoying. I guess this should be supported somehow. Changing title from "Form selection field selectedIndex not working properly" to "[4.xP]HTMLSelectElement.options.selectedIndex not supported". Since options is an HTMLCollection, this might require some work.
Grabbing this one.
Status: NEW → ASSIGNED
OS: Windows NT → All
Hardware: Other → All
Target Milestone: M11
Moving non-critical bugs out.
Moving to M14 see bug 16879 for reasons why.
*** Bug 23087 has been marked as a duplicate of this bug. ***
Setting seviarity to major.
Severity: normal → major
Setting the keyword all open [4.xp] bugs to 4xp.
Putting on PDT+ radar for beta1.
Summary: [4.xP]HTMLSelectElement.options.selectedIndex not supported → HTMLSelectElement.options.selectedIndex not supported
I brought the changes forward into my tree and they almost work (one bug I'm working on in setLength for bug 16879). Will update status later today. (Performance is better, but not great.)
Whiteboard: [PDT+] 11-Feb-2000 → [PDT+] 15-Feb-2000
Fix for this bug and bug 16879 is ready. I touched the following files (!=new) Will need a code review (I'll hit Vidur up for one) and approval. dom/public/idl/html/ ! nsHTMLOptionCollection.idl nsHTMLSelectElement.idl makefile.win Makefile.in dom/public/html/ nsIDOMHTMLSelectElement.h ! nsIDOMNSHTMLOptionCollection.h makefile.win Makefile.in MANIFEST dom/src/html/ nsJSHTMLSelectElement.cpp ! nsJSNSHTMLOptionCollection.cpp Makefile.in makefile.win (add to dom/macbuild/dom.mcp) layout/html/content/src/ Makefile.in makefile.win ! nsHTMLOptionCollection.cpp ! nsHTMLOptionCollection.h nsHTMLOptionElement.cpp nsHTMLSelectElement.cpp ! nsHTMLSelectElement.h (add to layout/macbuild/layout.mcp) layout/html/forms/src/ nsListControlFrame.cpp extensions/wallet/src/ wallet.cpp
Whiteboard: [PDT+] 15-Feb-2000 → [PDT+] Fix in hand, need code review / approval
Got a review from Vidur for this yesterday.
Whiteboard: [PDT+] Fix in hand, need code review / approval → [PDT+] Fix in hand, need approval
Whiteboard: [PDT+] Fix in hand, need approval → [PDT+] approved/reviewed fix in hand (will check in after bug 28691)
Just checked in the fix. This also fixed bug 16879.
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
verified with 02-29-09.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.