19956 - (see 22274) Extra space below image in cell

Status: VERIFIED DUPLICATE of bug 22274

(Core :: Layout: Tables, defect, P3)

Description

[richard]

In a three-row by one-cell table, the first and last rows containing an image
and the middle containing text, the first row is rendered with an extra
one-pixel buffer between it and the second row. The table is marked for
cellpadding, cellspacing, and border of 0, and the image (sized) is set for
vspace=0 border=0, to no avail.

Comment 1

[Mats Palmgren (inactive)]

Attachment: XHTML testcase

Comment 2

[Mats Palmgren (inactive)]

Attachment: HTML testcase

Comment 3

[Mats Palmgren (inactive)]

The XHTML testcase shows the bug; after removing the DOCTYPE it works fine, see
HTML testcase. (Using build 2000022916 on NT4.0 sp5)

Comment 4

[dcone (gone)]

I dumped the content models and they are the same between 3030 and 3029.

Comment 5

[dcone (gone)]

This may be correct, Pierre you should make sure.  The XHTML makes it run in
non-quirk mode... so the space that gets put into the table is a quirk relied
upon in a quirky Netscape Nav 4.x.  Peter Linss is the authority.

Comment 6

[Pierre Saslawsky]

Moving table bugs to M15 just to see how many we have.

Comment 7

[ekrock's old account (dead)]

Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!

Comment 8

[Pierre Saslawsky]

Block-moved to attinasi the bugs related to style inside tables

Comment 9

[Marc Attinasi]

This looks like a table problem. The style is correct (as seen using DumpStyle): 
the padding is set to 0 on all of the cells, as it should be. 

Using the Visual Debugging menu item in Viewer I can see that the frame that 
contains the image is smaller than the table cell, and the difference is the gap 
this bug is complaining about.

The problem appears when the DTD is set to STRICT mode. In TRANSITIONAL mode 
there is no problem.

I am adding another testcase that is essentially the same as the previous ones, 
however it has a bgcolor set on the cells so it is easier to see that the 
padding is the problem (the color shows up when the DTD is set to STRICT). By 
changing the DTD from 'STRICT.dtd' to 'TRANSITIONAL.dtd' you can see the 
behavior change.

Comment 10

[Marc Attinasi]

Attachment: Testcase showing padding problem using STRICT dtd

Comment 11

[karnaze (gone)]

mass move to m16

Comment 12

[Mark Tranchant]

Testcase site: http://www.billericaybaptistchurch.freeserve.co.uk/. Look in a 
couple of hours - the site currently doesn't render due to an xml declaration 
that I'll remove shortly.

This uses the strict DTD and the table "surrounds" are padded to the size of a 
character (it seems). Take away the DOCTYPE and it renders fine. Another 
alternative is to assign a class to affected cells that sets the font-size to 
1px - that also works, backing up my "character size" guess.

Comment 13

[karnaze (gone)]

Buster in the 3rd attachment (standard mode), the line inside the cell's block 
is 2 pixels taller than the image. 

Comment 14

[buster]

I'm 90% sure this is a dup of 22274.
David, looks like this is a line height issue in standard mode.  Could you take 
a look?

Comment 15

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

This is correct according to (our interpretation of) the CSS2 inline box model.
 (The "interpretation" is that CSS2 10.8.1 says that 'line-height' on a block
specifies the minimum height of all of that block's line boxes.  It doesn't say
*how* it sets this minimum.  If it's just a direct setting of a minimum, it
leads to really strange things, like the problem described here happening for
small images but not for large ones.  If it sets the minimum by creating an
anonymous inline with the font/line-height of the block, things make much more
sense (this is our interpretation).)

Comment 16

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

*** This bug has been marked as a duplicate of 22274 ***

Comment 17

[Katcha Da]

Verified duplicate.

Comment 18

[Daniel Wang]

housekeeping

Comment 19

[Pulsebot]

Pushed by ffxbld-merge:
https://hg.mozilla.org/releases/mozilla-beta/rev/d935f23fe203
[fenix] Issue https://github.com/mozilla-mobile/fenix/issues/19956: Add telemetry for tab view setting changes