140657 - Adjacent select tags combine into single select object

Status: RESOLVED FIXED

(Core :: DOM: HTML Parser, defect, P3)

Description

[Chris Dawes]

When this page is rendered it should render a select for each <SELECT> tag,
however it combines all of the adjacent form fields of the same name into one
select of with all the values within it. In other browsers it renders them
correctly as separate select lists.

I found this when I was writing a script that output the wrong thing, which
looked fine until I tested in other browsers (hehe).

Comment 1

[Chris Dawes]

Attachment: Here is the test html to prove the bug

here's the .html file that is referred to in the url

Comment 2

[Andrew Schultz]

seeing the same on linux build 20020426, also back to 0.9.7

==> HTML Form Controls

it seems like this probably done on purpose.  Certainly, on the testpage, it is
more useful to combine the elements into single selects.

Comment 3

[Boris Zbarsky [:bzbarsky]]

To parser.  You have no </select> tags and it looks like we don't close out
<select> elements automatically.... (note, per HTML4 we should not close them out).

Comment 4

[Chris Dawes]

It works correctly with the </SELECT> tags inserted:

As you can not nest <SELECT> tags, wouldn't it be sensible to close the previous
tag? I would assume this would occur for all non-nestable tags?

Please let me know if this is redundant thinking? I though this would be logical?

I still think that this is a bug.
I also think the argument is valid, even though it relies on bad code to prove
the point. 

Comment 5

[harishd]

Yes, for backwards compatibility we should probably autoclose SELECT.

Comment 6

[harishd]

This bug has been marked "future" because the original netscape engineer working 
on this is over-burdened. If you feel this is an error, that you or another
known resource will be working on this bug,or if it blocks your work in some way
 -- please attach your concern to the bug for reconsideration. 

Comment 7

[Boris Zbarsky [:bzbarsky]]

Similar issues to bug 171713 

Comment 8

[Boris Zbarsky [:bzbarsky]]

Fixed by checkin for bug 171713.