Closed Bug 2452 Opened 26 years ago Closed

One word is way too low

Categories

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

x86
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: dbaron, Assigned: rickg)

References

()

Details

The word "Reporter" shows up about 10 lines below where it should be.
Assignee: troy → rickg
Component: Layout → Parser
Parser isn't handling the tables correctly. Here's a small HTML fragment that
demonstrates the problem:

<FORM NAME=queryForm METHOD=GET ACTION="buglist.cgi">
<TABLE>
<TR><TD>Reporter:<TD><INPUT NAME="reporter" SIZE=45 VALUE="">
</TABLE>
<table>
<tr>
<TH ALIGN=LEFT>Program:</th>
</tr>
</table>

The key is that there's no </tr> to terminate the row of the first table. If you
add a </tr> then it displays fine

Here's what the content model looks like:
webshell=00F1B6F0
HTML refcount=6<
  HEAD refcount=2<
  >
  BODY refcount=3<
    FORM NAME=queryForm METHOD=get ACTION=buglist.cgi refcount=5<>
    Text refcount=3<\n\n>
    TABLE refcount=7<
      TBODY refcount=3<
        TR refcount=3<
          TD refcount=4<
            Text refcount=3<Reporter:>
          >
          TD refcount=4<
            INPUT NAME=reporter SIZE=45 VALUE= refcount=4<>
            Text refcount=3<\n\n\n>
            TABLE refcount=6<
              TBODY refcount=3<
                TR refcount=3<
                  TH ALIGN=left refcount=4<
                    Text refcount=3<Program:>
                  >
                >
              >
            >
            Text refcount=3<\n\n>
          >
        >
      >
    >
  >
>

Notice that we ended up with nested tables which is what causes the problem...
Status: NEW → ASSIGNED
Fixed by improvements to new DTD system; also fixed one in comment parsing.
QA Contact: 3881
dbaron, can you mark this VERIFIED_FIXED please??
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.