cell padding depends from table nesting for table-layout:fixed

VERIFIED FIXED in mozilla0.9

Status

()

Core
Layout: Tables
P3
normal
VERIFIED FIXED
18 years ago
17 years ago

People

(Reporter: Matthias Kerkhoff, Assigned: karnaze (gone))

Tracking

({css1, testcase})

Trunk
mozilla0.9
x86
Windows NT
css1, testcase
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(7 attachments)

(Reporter)

Description

18 years ago
From Bugzilla Helper:
User-Agent: Mozilla/4.73 [en] (WinNT; U)
BuildID:    2000070320

Tables using the fixed layout algorithm (table-layout:fixed) seem to ignore the 
padding on all cells in a row (except on the first one).

Reproducible: Always
Steps to Reproduce:
1. Create a table with one row and some cells
2. Set css widths and paddings for each cell
3. Load the table

Actual Results:  The paddding is ignored for all cells except the first one.

Expected Results:  The padding should be applied to all cells where it has been 
specified.							

padding works fine when used with other table layout algorithms.
(Reporter)

Updated

18 years ago
Keywords: css1
(Reporter)

Comment 1

18 years ago
Created attachment 10965 [details]
testcase for 44505 and 44506
(Reporter)

Comment 2

18 years ago
Bug seems to be related to nested tables. Couldn't reproduce with plain tables.
Keywords: testcase
(Reporter)

Comment 3

18 years ago
Created attachment 10966 [details]
Corrected testcase

Comment 4

18 years ago
Created attachment 11102 [details]
reduced testcase

Comment 5

18 years ago
The padding seems to depend on the table nesting (see reduced testcase).
Confirming the bug. Changing summary
Status: UNCONFIRMED → NEW
Ever confirmed: true
Summary: padding is ignored on all but the first cell if table-layout:fixed → cell padding depends from table nesting for table-layout:fixed
(Assignee)

Updated

18 years ago
Status: NEW → ASSIGNED
Target Milestone: --- → M19

Updated

18 years ago
QA Contact: desale → chrisd

Comment 6

18 years ago
Created attachment 24381 [details] [diff] [review]
patch

Comment 7

18 years ago
Created attachment 24382 [details]
testcase

Comment 8

18 years ago
Created attachment 24383 [details]
testcase with image

Comment 9

17 years ago
Created attachment 25178 [details] [diff] [review]
revised patch (new style handling)

Comment 10

17 years ago
Finally, I managed to verify that the patch does not not break the table
regression tests. (It would be also very difficult, as we do not have any
testcase in the regression tests that tests for clipping ... this is a ping to
the QA engineer). The patch enables the overflow:hidden from CSS2 for table
cells. I think with the correct handling of the padding we outperfom with this
patch IE5. They have overflow:hidden for the fixed layout (you cannot make it
visible) and they do not handle padding correctly. This patch does not enable
overflow:scroll and overflow:auto, but I think it improves anyway our CSS2
compliance.
Keywords: patch, review

Comment 11

17 years ago
sr=attinasi - the patch looks fine. Can we add the testcase to the table
regression tests? Generally, Chris K. adds them and not a QA person. As you
mentioned, Bernd, we need a testcase in the regression suite to catch this case
and none currently exist. Thanks.

Comment 12

17 years ago
> Generally, Chris K. adds them and not a QA person.

But should this prevent chrisd@netscape.com to write something comparable to
marvin for CSS2 table stuff, I think we need it desperately.

Comment 13

17 years ago
The testcase renders now fine. fix checked in
Status: ASSIGNED → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 14

17 years ago
Please notice that the even the corrected testcase has an omitted padding
attribute for the third cell. 
	<TD nowrap style="text-align:left; 0px 5px 0px 5px; white-space:nowrap;
width:45px">45px</TD>
(Assignee)

Updated

17 years ago
Target Milestone: --- → mozilla0.9

Comment 15

17 years ago
QA contact update
QA Contact: chrisd → amar

Comment 16

17 years ago
 The attached testcase renders fine
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.