72198 - table width calculation doesn't account for border width

Status: RESOLVED DUPLICATE of bug 71850

(Core :: Layout, defect)

Description

[Kathleen :Brade]

It appears that when calculating the layout width of a table (that is 100%), 
somewhere we are only accounting for one half of the border (the left or right 
side but not both).  With a table at 100% width of window, I see a scrollbar 
appear.

I expect that the scrollbar shouldn't appear (and the entire border be visible) 
OR that the scrollbar appears and the width of the table (excluding all border) 
is 100% and the border * 2 is the extra distance to be scrolled.

Right now, the table seems to be following the CSS spec
(http://www.w3.org/TR/REC-CSS2/tables.html#)

Comment 1

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

Attachment: testcase where we're not following the CSS spec

Comment 2

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

Relevant quote from CSS2 is
http://www.w3.org/TR/REC-CSS2/tables.html#collapsing-borders in the paragraph
right after the figure:

  Note that in this model, the width of the tabel includes half the table border.

We seem to be *moving* the width so we can fit all of the border in the
containing block, and the total overflow width is also 2*border-width instead of
1*border-width. (see above testcase)

Comment 3

[rubydoo123]

just an fyi, using the build from 2001031204 on win98, I did not see this 
behavior, I downloaded today's build (20010304) and I see the issue that Kathy 
has raised.

Comment 4

[karnaze (gone)]

brade, if you are commenting on tables in the separate border model, then I am 
not seeing the problem you describe in the following:

<table border width=100%> 
 <tr> 
  <td>foo</td> 
 </tr> 
</table>

If you have a better example, please mark this as m0.9, because it represents a 
serious regression.

If you are referring to the collapsing border model (as dbaron's example 
illustrates) then that has been turned off for quite a while. Assumming the 
later, I'm marking this m1.0 to coincide with the new collapsing border code. 

Comment 5

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

Oops, I forgot the CBM was turned off.

So maybe this is just a dup of bug 71850?

Comment 6

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

Yeah, it is.  The fix fixes my testcase and I'm it would also explain the
problem brade was seeing.

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