37618 - font tag in option closes out select

Status: VERIFIED FIXED

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

Description

[Duc Tang]

There is a problem with tbe nowrap. This may be a dupe of 18644 though.

Comment 1

[Doron Rosenberg (IBM)]

I'm seeing this too. Removing all the NOWRAP makes the page load normal. The
page renders correct in 4.7x, adding 4xp keyword. It also works with Netscape pr1.

Confirming bug.

Comment 2

[Peter "jag" Annema]

The nowrap isn't the problem, it actually shows it. The "Select a State" box has
its options shown as text, inside a <td></td>. It's a long text, all those
states, so you get a very wide column in your table.

The actual problem is in these lines:
<form name="frmStates" method="post"><br><img src="/ci/nat_s.gif"
alt="National"><br><font face="arial" size="2">
<select name="selectState" width="20"
onChange="location.href=frmStates.selectState.options[frmStates.selectState.options.selectedIndex].value">
<option value=""><Font size="2" face="arial">Select a State</font>

Remove the first (unclosed) font tag and the page should display fine. I'll
creating a simplified case.

Should Mozilla handle this more gracefully, or should it draw the line "fix yer
html"?

Changing component to HTML Form Controls.

Comment 3

[Peter "jag" Annema]

Attachment: Shows problem with unclosed tag and <select> <option>

Comment 4

[karnaze (gone)]

Rod, the attachment doesn't have a table in it.

Comment 5

[rods (gone)]

<select>
<option><font>Option 1</font>
<option>Option 2
<option>Option 3
</select>

Option content should never have font tags in them.

The parser seems to be doing the correct thing by closing out the select when it 
hits the "font" tag. But for nav quirks we need it to ignore the font tag and 
continue processing the options.

Comment 6

[rods (gone)]

changed summary

Comment 7

[Peter "jag" Annema]

That's because the problem isn't in tables...

I was about to fix the summary, but someone beat me to it:
<font> around <select> with <font> in <option> breaks select

Comment 8

[Peter "jag" Annema]

Rod is almost correct, the problem however lies in the <font> tag before the 
<select>:

<form method="post">
<font face="arial" size="2">    <!-- this one -->
<select>
<option><font>Option 1</font>
<option>Option 2
<option>Option 3
</select>
</form>

If you remove the first <font> it will render just fine in Mozilla. Rod's 
example is actually the second (working) example in the attachment.
Shouldn't that first <font> be closed? Or has this to do with bug 6169 where 
NavQuirk support for <font><select></font> has been put in?

Comment 9

[harishd]

Reassigning to myself.

Comment 10

[harishd]

*** Bug 38377 has been marked as a duplicate of this bug. ***

Comment 11

[rods (gone)]

*** Bug 39767 has been marked as a duplicate of this bug. ***

Comment 12

[rods (gone)]

This completely horks any pages that have font tags or any other type of tag 
buried inside a select. I see this more and more. Nominating for nsbeta3

Comment 13

[rods (gone)]

*** Bug 42597 has been marked as a duplicate of this bug. ***

Comment 14

[rods (gone)]

*** Bug 43072 has been marked as a duplicate of this bug. ***

Comment 15

[harishd]

Attachment: Proposed patch

Comment 16

[rods (gone)]

*** Bug 44849 has been marked as a duplicate of this bug. ***

Comment 17

[harishd]

*** Bug 45537 has been marked as a duplicate of this bug. ***

Comment 18

[Nisheeth Ranjan]

Marking nsbeta3+.  Harish please check in your fix.  Thanks.

Comment 19

[harishd]

Fix is in.

Comment 20

[Christine Hoffman]

Tested using 8/25 builds on Win and Mac and 8/28 build on Linux. Verified bug 
fixed (assuming FONT FACE has no effect on option text).