167408 - padding on inline makes table cell overflow (incorrect max-element-width)

Status: RESOLVED FIXED

(Core :: Layout: Block and Inline, defect, P2)

Description

[Foux]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.1) Gecko/20020826
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.1) Gecko/20020826

In the page http://www.foux.net/CSS_padding_bug.html, the padded text in the
second cell of the table, is displayed beyond the table limits. When text
padding settings of the <B> style are removed, the text is displayed correctly.

Reproducible: Always

Steps to Reproduce:
1. http://www.foux.net/CSS_padding_bug.html


Actual Results:  
The text in the second cell of the table is displayed beyond the table limits.

Expected Results:  
The text in the second cell should not exceed the table limits.

Comment 1

[Boris Zbarsky [:bzbarsky]]

I suspect the padding is not included in the MES... or this is a tables bug. 
bernd?  What do you think?

Comment 2

[Bernd]

this might be a dupe of bug 109927

Comment 3

[Bernd]

Attachment: testcase

Comment 4

[Bernd]

It looks like this is problem with inline frames MES computation

      cell 02EDC13C r=0 a=UC,UC c=UC,UC cnt=865 
       block 02EDC19C r=0 a=UC,UC c=UC,UC cnt=866 
        inline 02EDC360 r=0 a=UC,UC c=UC,UC cnt=867 
         text 02EDC3CC r=0 a=UC,UC c=UC,UC cnt=868 
         text 02EDC3CC d=3000,285 me=3000 
        inline 02EDC360 d=4500,285 me=3000 
       block 02EDC19C d=4500,300 me=3000 
      cell 02EDC13C d=4560,360 me=3060 

Comment 5

[Bernd]

Attachment: patch

Comment 6

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

Comment on attachment 105208 [details] [diff] [review]
patch

Shouldn't you only add the right border if the border is adjacent to the chunk
of text that's causing the MES?  And doesn't the MES computation need to span
inline frames anyway, to handle things like

<span>foo</span>bar

?  What about the case where that span has a border, and you need to get the
correct value for both:

<span>a aoeuidhtns b</span>c

and

<span>abc</span>def

?

Comment 7

[Bernd]

Attachment: testcase

testcase with different scenarios as proposed by David

Comment 8

[Bernd]

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

Comment 9

[Boris Zbarsky [:bzbarsky]]

To correct component.

Comment 10

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

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

Comment 11

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

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

Comment 12

[Boris Zbarsky [:bzbarsky]]

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

Comment 13

[keefedm]

I've encountered this with an anchor tag in a table cell.  In my case (under
both 0.8 and 0.9.3), it seems to correctly honor left/right padding, but
top/bottom padding spills beyond cell boundaries.  Interestingly, IE also
correctly honors left/right, but clips top/bottom (rather than spilling over or
expanding cell).

Comment 14

[Erik Fabert]

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

Comment 15

[Jennifer Grucza]

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

Comment 16

[Boris Zbarsky [:bzbarsky]]

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

Comment 17

[Boris Zbarsky [:bzbarsky]]

Fixed by reflow branch.