102370 - offset of the DT in the DL descriptive list is off

Status: RESOLVED FIXED

(Core :: Layout, defect, P2)

Description

[Ronald Landheer]

From Bugzilla Helper:
User-Agent: Mozilla/5.0 (Windows; U; Win98; en-US; rv:0.9.4) Gecko/20010913
BuildID:    2001091303

All <DT> tags should be aligned equally (to the left) on this page - they
aren't: the first one is, the second one is from time to time, the third one is
always aligned more to the right.
The <DD> stuff is aligned correctly

Reproducible: Always
Steps to Reproduce:
1. open the page
2. look
3. see?

Actual Results:  Alignment was wrong - from the web, the second was too far to
the right. Locally, it was the third one that was too far to the right.

Expected Results:  See above (alignment should be on the left of the viewport)

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 2

[Mats Palmgren (inactive)]

It's the unclosed <A> elements that is causing the problem. See also bug 100930.

Comment 3

[Marc Attinasi]

Yes, the unclosed <A> is causing residual style to change the indenting level of
the DD's. Here is the content model of the testcase from mats:

html@02EBCE00 refcount=8<
  head@02EBCD10 refcount=2<
    title@017406B0 refcount=2<
      Text@01740650 refcount=2<Testcase for bug 102370>
    >
  >
  body@017404A0 refcount=3<
    Text@02EDA7A0 refcount=3<\n\n\n>
    dl@02EDA680 refcount=3<
      a@02EDA520 name=a1 refcount=4<
        Text@02EDA3D0 refcount=3<\n    >
        dt@02EDA2A0 refcount=3<
          Text@02EDA240 refcount=3<First DT\n    >
        >
      >
      dd@02EDA030 refcount=3<
        a@02EDBE80 name=a1 refcount=3<
          Text@02EDBCD0 refcount=3<First DD\n  \n  >
        >
        a@02EDBC20 name=a2 refcount=4<
          Text@02EDBB10 refcount=3<\n    >
          dt@02EDBA70 refcount=3<
            Text@02EDBA10 refcount=3<Second DT\n    >
          >
        >
      >
      dd@02EDB870 refcount=3<
        a@02EDB800 name=a2 refcount=3<
          Text@02EDB5E0 refcount=3<Second DD\n>
        >
      >
    >
    Text@02EDB4A0 refcount=3<\n\n\n>
  >
>
Over to Harish to look at. I don't think it is directly related to bug 100930,
but it might be.

Comment 4

[harishd]

Attachment: Patch v1.0 [ DT should not close DD and vice versa ]

Comment 5

[Heikki Toivonen (remove -bugzilla when emailing directly)]

Comment on attachment 53309 [details] [diff] [review]
Patch v1.0 [ DT should not close DD and vice versa ]

r=heikki

Normal - and bitwise | in the same expression looks kinda funny though...

Comment 6

[Johnny Stenback (:jst)]

Comment on attachment 53309 [details] [diff] [review]
Patch v1.0 [ DT should not close DD and vice versa ]

sr=jst

Comment 7

[harishd]

fixed.

Comment 8

[Christopher Hoess (gone)]

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