54193 - quirk: CSS line height set to 0 causes next element to overlap

Status: NEW

(Core :: Layout, defect, P4)

Description

[buster]

using debug build from branch 9/24/00 WinNT.
broken out from bug 52670
in the to-be-attached test case, we are respecting the request for line-height:0.
Nav4 and IE 5.5 for Windows do not do this.  They use the default line height.

Comment 1

[buster]

Attachment: reduced test case

Comment 2

[buster]

david, ian: are we rendering this correctly?  I think so.

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I'm not sure.  It's probably not correct for strict mode.  It needs to be looked
into in detail regarding quirks.

But there was another bug somewhere with a very similar test case... (perhaps I
even own it...)

Comment 4

[Hixie (not reading bugmail)]

Correct behaviour is:

                      +
   September 26, 2000 | 
   +----------------+ +
   +------------------------------+
   | ____________________________ |
   +------------------------------+

...where
   +---+ 
   |   | = a line box (not visible)
   +---+

   _____ = the image

Thus in strict mode we are correct. In quirks mode... well, who cares, quirks
mode is quirky and non-standard anyway, and who uses line-height:0 in legacy
documents? It has never worked in browsers before...

Strongly recommend WONTFIX. Marking as such.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I'd like to try to figure out why the old browsers do what they do.  But not 
now...

Reopening...

Comment 6

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

...and assigning to myself, P4/minor/Future.

buster:  If you want to investigate sooner, feel free to take this back.

Comment 7

[Hixie (not reading bugmail)]

BTW, correction to my comments: 
   +---+ 
   |   | = the bounds of an inline box (not visible)
   +---+
...not a line box as I stated above.

Comment 8

[Hixie (not reading bugmail)]

Upon managerial request, adding the "testcase" keyword to 84 open layout bugs that
do not have the "testcase" keyword and yet have an attachement with the word
"test" in the description field. Apologies for any mistakes.

Comment 9

[Thomas]

From comment #4 I understand "Strongly recommend WONTFIX. Marking as such."?