Closed Bug 253701 Opened 21 years ago Closed 18 years ago

Table column appears twice or three times

Categories

(Core :: Layout: Tables, defect)

x86
Linux
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: osavill, Unassigned)

References

()

Details

(Keywords: testcase)

Attachments

(3 files)

User-Agent: Mozilla/5.0 (compatible; Konqueror/3.2; Linux) (KHTML, like Gecko) Build Identifier: http://www.sterlingxs.co.uk/ and http://www.sterlingxs.co.uk/scpages/newlaptops.html both show odd behaviour when it comes to rendering the left hand side. There appears to be duplication of the left hand edge of the main display area. This duplicated area disappears when the browser window is made small enough. http://www.sterlingxs.co.uk/scpages/laptopaccessories.html does not display this behaviour. Reproducible: Always Steps to Reproduce: 1. 2. 3.
That table cell has a width of "149", but we're rendering it a lot wider than that, so the background tiles...
Assignee: general → nobody
Component: Browser-General → Layout: Tables
QA Contact: general → core.layout.tables
Need a minimal testcase...
Keywords: qawanted
Attached file Testcase (deleted) —
We have the same layout all the way back to and including Mozilla 1.0.2 It sure seems like a bug to me though.
Keywords: qawantedtestcase
Attached file Testcase #2 (standards mode) (deleted) —
The problem does not occur in Standards mode...
Bernd, it looks like the quirky code in question is at http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/layout/html/table/src/BasicTableLayoutStrategy.cpp&rev=3.225&mark=1071-1077#1070 It looks like this is blowing away all the specified widths, not just that for the cell in the first row?
I think bug 269156 is the same bug. Bernd is pointing at the same lines of code there as Boris.
Yeah. The patches there will probably fix this bug, as well as dozens of other cell-sizing bugs we have on file....
Depends on: 269156
WFM with Minefield 3a1. likly fixed with reflow bug 3000330.
Bug occurs in SeaMonkey 2006120701 (pre reflow branch) on Linux. Bug does not occur in SeaMonkey 2006120801 (post reflow branch) on Linux. -> FIXED (by the reflow branch landing)
Status: UNCONFIRMED → RESOLVED
Closed: 18 years ago
Depends on: reflow-refactor
Flags: in-testsuite?
Resolution: --- → FIXED
Attached patch Reftests (deleted) — Splinter Review
Attachment #257621 - Flags: review?(dbaron)
Comment on attachment 257621 [details] [diff] [review] Reftests Checked in after testing in pre- and post-reflow-branch builds.
Attachment #257621 - Flags: review?(dbaron) → review+
Status: RESOLVED → VERIFIED
Flags: in-testsuite? → in-testsuite+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: