Closed Bug 3992 Opened 26 years ago Closed 25 years ago

'height' style property does not work with TABLE, TR, TBODY, THEAD or TFOOT elements. It DOES work with TD element

Categories

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

All
Windows 95
defect

Tracking

()

VERIFIED FIXED

People

(Reporter: christinehoff4, Assigned: karnaze)

Details

(Keywords: css2, relnote, testcase, Whiteboard: [nsbeta3+][nsbeta2-])

Attachments

(11 files)

Using 3/18 build on Win 95, Win 98, Win NT, Mac8.5 and Linux. The following test cases demonstrate the problem: TABLE: http://slip/projects/marvin/bugs/table_height.html (does not work) TD: http://slip/projects/marvin/bugs/td_height.html (OK - works fine) TR: http://slip/projects/marvin/bugs/tr_height.html (does not work) TBODY: http://slip/projects/marvin/bugs/tbody_height.html (does not work) THEAD: http://slip/projects/marvin/bugs/thead_height.html (does not work) TFOOT: http://slip/projects/marvin/bugs/tfoot_height.html (does not work) In the test cases above, a height value of 80pt has been assigned to the individual element indicated. NOTE - will write up as a separate bug: The 'height' property value assigned to the TABLE element behaves differently on different platforms.
Status: NEW → ASSIGNED
The CSS2 rules on the height property in tables are at: http://www.w3.org/TR/REC-CSS2/tables.html#height-layout I don't think the spec says anything about handling height on table-row-group, table-footer-group, or table-header-group
Target Milestone: M6
Moving to M7.
The given test case URLs do not work for me. Are there alternatives?? BTW, this bug still seems to be set for M6, though the last comment says it would be in M7...
Moving to M8.
Whiteboard: [TESTCASE]
Marked with [TESTCASE].
Summary: CSS 'height' style property does not work with TABLE, TR, TBODY, THEAD or TFOOT elements. It DOES work with TD element → {css2} 'height' style property does not work with TABLE, TR, TBODY, THEAD or TFOOT elements. It DOES work with TD element
I'm marking this {css2}, because although 'height' is in CSS1, tables are not. Implementing this correctly involves using the CSS2 definitions.
I think height is working on TABLE, but not TR. It's hard to tell in the testcase, because 80pt is not much bigger than the height it would otherwise get. Perhaps the testcases should be revised. The correct behavior when multiple heights (or widths) are specified is probably quite complicated - and height and width on many table elements should be taken as min-height or min-width (depending on the situation, especially for height).
Moving to M9.
Attached file Test case for TD element (this works) (deleted) —
Target Milestone: M9 → M12
See http://www.fas.harvard.edu/~dbaron/css/test/sec170503 for additional test cases. Bug 2886 (with fix attached) and bug 10207 also cause problems on that page.
mass move to m14.
Keywords: css2
Migrating from {css2} to css2 keyword. The {css1}, {css2}, {css3} and {css-moz} radars should now be considered deprecated in favour of keywords. I am *really* sorry about the spam...
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Target Milestone: M14 → M15
Adding 'beta1' keyword.
Keywords: beta1
Whiteboard: [TESTCASE]
Marking PDT- because it's CSS2.
Whiteboard: [PDT-]
Summary: {css2} 'height' style property does not work with TABLE, TR, TBODY, THEAD or TFOOT elements. It DOES work with TD element → 'height' style property does not work with TABLE, TR, TBODY, THEAD or TFOOT elements. It DOES work with TD element
mass move to m16
Target Milestone: M15 → M16
Heights and widths don't work on table cells, either, when they are less than the contents' dimensions.
That's certainly the way it should work for 'width', and probably for 'height' as well. See http://www.w3.org/TR/REC-CSS2/tables.html#auto-table-layout . Essentially, 'height' and 'width' on table elements are treated as 'min-height' and 'min-width', by tradition.
PDT- from beta1 removed; nsbeta2 added
Keywords: nsbeta2
Whiteboard: [PDT-]
Per ekrock mail: 'This is a bug but not a beta2 stopper. Could fix for b3.' Putting on [nsbeta2-] radar and adding nsbeta3 keyword.
Keywords: nsbeta3
Whiteboard: [nsbeta2-]
Moving to M17.
Target Milestone: M16 → M17
Keywords: relnote
The only heigths that can be set are table, row, and cell and only the row is not working. I'm attaching a patch for that in case it is lost before the tree opens.
Whiteboard: [nsbeta2-] → [nsbeta2-] fix-in-hand
Attached file patch for row height support (deleted) —
Attached file a better patch, ignore the last one (deleted) —
Approving for nsbeta3
Whiteboard: [nsbeta2-] fix-in-hand → [nsbeta2-] [nsbeta3+] fix-in-hand
Priority: P3 → P2
I will review the patch and if ok, check it in.
Assignee: karnaze → buster
Status: ASSIGNED → NEW
checking in fix for chris. r=buster
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [nsbeta2-] [nsbeta3+] fix-in-hand → [nsbeta3+]
Using the following platforms/builds: Win 98: 8/25/06 Mac 8.6: 8/25/04 Linux Red Hat 6.0 (version 2.2): 8/25/08 bug still not fixed for TABLE, TBODY, THEAD and TFOOT. Problems: TABLE: height is not honored, and tfoot and tbody are in reverse order THEAD: height is not honored TBODY: height is not honored TFOOT: height is not honored Testcases attached. Reopening bug. Should these be written up as separate bugs?
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attached file Height on TABLE (deleted) —
Attached file height on TABLE (deleted) —
Attached file height on TBODY (deleted) —
Attached file height on THEAD (deleted) —
Attached file height on TFOOT (deleted) —
Adding [nsbeta2-] since PR2 is out and [nsbeta3+] is already indicated.
Whiteboard: [nsbeta3+] → [nsbeta3+][nsbeta2-]
I had marked this fixed based on karnaze's comment: ------- Additional Comments From karnaze@netscape.com 2000-05-26 14:20 ------- The only heigths that can be set are table, row, and cell and only the row is not working. I'm attaching a patch for that in case it is lost before the tree opens. ----------------------------------------------------------------------------- To me, this implies that Chris thought it was illegal to apply height to a rowgroup (THEAD, TFOOT, TBODY). The fix was just for TR. So I'll leave it to Karnaze to decide what should be done with rowgroups.
Assignee: buster → karnaze
Status: REOPENED → NEW
If you can apply 'height' style to TR, I don't understand why you can't apply it to row-groups. I don't find any restriction in the specs. At any rate, applying height to TABLE is still broken. I will followup with Karnaze upon his return.
ChrisD, I'm marking this fixed since Buster checked in the patch to make height on <tr> work. If you think that it should work on <tbody>, <thead>, <tfoot>, please open a new bug and mark it "future" because it is not a very high priority and the spec is unclear on it.
Status: NEW → RESOLVED
Closed: 25 years ago25 years ago
Resolution: --- → FIXED
TR problem fixed. Remaining issues: (1) When height is assigned to TABLE element, TBODY and TFOOT are reversed (2) Height still does not work on THEAD, TBODY, and TFOOT. Closing this nsbeta3+ bug out as crucial elements are working (TABLE, TR, TD). Will write up separate bugs for (1) and (2) issues.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: