234593 - Growing table

Status: RESOLVED FIXED

(Core :: Layout: Tables, defect)

Description

[u61093]

User-Agent:       
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7a) Gecko/20040212 Firebird/0.8.0+

Go to the URL and mouse over the "Document Literal" link that is next to the
text "Wiki:".  The center cell on the page grows larger and larger to the right
as you mouse over the link.

Reproducible: Always
Steps to Reproduce:
See details.
Actual Results:  
See details.

Expected Results:  
The size of the center cell should remain constant.

Comment 1

[Charles Fenwick]

not specific to Firefox

Comment 2

[u61093]

I'm not sure what "not specfic to firefox" means, but it doesn't happen on IE6.
 Perhaps the poster was referring to Mozilla, Firebird, or Phoenix?

Comment 3

[Alex Eng]

Reproduced on build 2004030108
Mozilla/5.0 (Windows; U; Win98; en-US; rv:) Gecko/20040301

Confirming and moving to Layout: Tables

terris: Yes, Charles Fenwick was referring to Mozilla and Firefox (formerly
Firebird, and *sigh*, formerly Phoenix).  Because Firefox shares the same Gecko
layout engine as Mozilla, if you run into a layout bug in Firefox, chances are
it affects Mozilla as well.

Comment 4

[Martijn Wargers (dead)]

I can't find any "Document Literal" link on that page, and I can't make it to
grow the table, using:
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8a4) Gecko/20040907
Firefox/0.9.1+

But I see a problem with the page. On first load or shift->reload I see that the
middle column is too wide, which is fixed on a regular reload.

Comment 5

[Martijn Wargers (dead)]

Attachment: Simplified testcase using Hyatt's hack

This is a simplified testcase of the bug I described in comment 4.
You should see no horizontal scrollbar in this testcase (but there is), the
table should not be that wide.
I think the I described is more or less the same bug as bug 238856.

Comment 6

[Bernd]

the fr.... xul frame returns the previously available width as mew

Comment 7

[Bernd]

Attachment: patch

Comment 8

[Bernd]

Robert, should I add the same code the the XUL scroll frame? I don't know when
it is executed.

Comment 9

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Yes, add it to the XUL scroll frame. They should stay in sync at this point.

Comment 10

[Bernd]

Attachment: patch for both frame types

Comment 11

[Bernd]

Comment on attachment 161663 [details] [diff] [review]
patch for both frame types

switching reviewer (Robert is busy)

Comment 12

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

Is this really the right thing to do for 'auto' width?  Wouldn't this break
something like:

<table width="1" border><tr><td>
  <div style="overflow:hidden">Hello</div>
</td></tr></table>

Comment 13

[Bernd]

Attachment: davids comment as a testcase

Comment 14

[Bernd]

No the patch does not alter the rendering in this case, you will not see the
"Hello". The rendering is identical to Opera but is different from IE.

Comment 15

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

Comment on attachment 161663 [details] [diff] [review]
patch for both frame types

r+sr=dbaron, but fix the odd indentation in the new code (one line
over-indented, although you could make both assignments into one, i.e.,
a=b=...)

Comment 16

[Bernd]

fix checked in

Comment 17

[Bernd]

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

Comment 18

[Bernd]

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

Comment 19

[Bernd]

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

Comment 20

[Hideo Saito]

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