60807 - Columns does not align (over-specified COLSPAN problem)

Status: RESOLVED FIXED

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

Description

[Mats Palmgren (inactive)]

Columns does not align (over-specified COLSPAN problem).

REPEAT-BY:
1. goto http://www.price-search.net/
2. Click on the "Digital Camera" link, taking you to this URL:
http://engine.bottomdollar.com/fastsearch?Type=Search&Template=BDN-898-SSE&WebSi
te=BDN-898-SSE&Category=Hardware&SearchFor=digital+camera

ACTUAL RESULTS:
In the search result list, the last two columns (Price & Info) does not align
with their respective headings.

PLATFORMS TESTED:
Bug occurs in Mozilla nightly build 2000-11-19-20-trunk on Windows 98 SE.

Bug does not occur in:
Netscape Navigator 4.74 on Windows 98 SE.
IE 5.0 on Windows 98 SE.
Opera 4.01 on Windows 98 SE.

Comment 1

[Mats Palmgren (inactive)]

Attachment: Testcase

Comment 2

[karnaze (gone)]

Fixed with latest checkin.

Comment 3

[Bernd]

fixed by Chris in December, broken by Chris in January (2001-01-13)

Comment 4

[Amarendra Hanumanula]

QA contact update

Comment 5

[karnaze (gone)]

Attachment: patch to fix the bug

Comment 6

[karnaze (gone)]

Attachment: correct patch (ignore the previous one)

Comment 7

[Bernd]

r = bernd, I applied the patch it works, it does not break 42443. It looks good,
very logical.

Comment 8

[Marc Attinasi]

Looks good to me - sr=attinasi

Comment 9

[Bernd]

>A good code reviewer can feel like your worst enemy

this patch breaks the table regression tests: 
core/colspan.html
core/colspan2.html see the third table from bottom
bug27993-2.html

removing my 'r' until these issues are solved
(one more thanks to Chris Waterson for making the regression tests a sharp tool
again)

Comment 10

[karnaze (gone)]

Attachment: patch to fix col_span col_span2 test cases (to be applied after attachment #3)

Comment 11

[karnaze (gone)]

Bernd, thanks for catching this. I'm not sure how I let this happen. Anyway, the 
new patch is in.