Status

()

Core
HTML: Parser
P2
normal
VERIFIED FIXED
20 years ago
19 years ago

People

(Reporter: dbaron, Assigned: rickg)

Tracking

Trunk
x86
Windows 95
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(URL)

(Reporter)

Description

20 years ago
The word "Reporter" shows up about 10 lines below where it should be.

Updated

20 years ago
Assignee: troy → rickg
Component: Layout → Parser

Comment 1

20 years ago
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...
(Assignee)

Updated

20 years ago
Status: NEW → ASSIGNED
(Assignee)

Comment 2

20 years ago
Fixed by improvements to new DTD system; also fixed one in comment parsing.

Updated

20 years ago
QA Contact: 3881

Comment 3

20 years ago
dbaron, can you mark this VERIFIED_FIXED please??
(Reporter)

Updated

20 years ago
Status: RESOLVED → VERIFIED
(Reporter)

Comment 4

20 years ago
Verified as fixed.
You need to log in before you can comment on or make changes to this bug.