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)
Tracking
()
VERIFIED
FIXED
M17
People
(Reporter: christinehoff4, Assigned: karnaze)
Details
(Keywords: css2, relnote, testcase, Whiteboard: [nsbeta3+][nsbeta2-])
Attachments
(11 files)
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
text/plain
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details | |
(deleted),
text/html
|
Details |
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.
Assignee | ||
Updated•26 years ago
|
Status: NEW → ASSIGNED
Comment 1•26 years ago
|
||
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
Assignee | ||
Updated•26 years ago
|
Target Milestone: M6
Assignee | ||
Comment 2•26 years ago
|
||
Moving to M7.
Comment 3•26 years ago
|
||
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...
Assignee | ||
Comment 4•26 years ago
|
||
Moving to M8.
Reporter | ||
Comment 5•26 years ago
|
||
Updated•26 years ago
|
Whiteboard: [TESTCASE]
Comment 6•26 years ago
|
||
Marked with [TESTCASE].
Updated•26 years ago
|
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
Comment 7•26 years ago
|
||
I'm marking this {css2}, because although 'height' is in CSS1, tables are not.
Implementing this correctly involves using the CSS2 definitions.
Comment 8•26 years ago
|
||
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).
Assignee | ||
Comment 9•26 years ago
|
||
Moving to M9.
Reporter | ||
Comment 10•26 years ago
|
||
Reporter | ||
Comment 11•26 years ago
|
||
Assignee | ||
Updated•26 years ago
|
Target Milestone: M9 → M12
Comment 12•26 years ago
|
||
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.
Assignee | ||
Comment 13•25 years ago
|
||
mass move to m14.
Comment 14•25 years ago
|
||
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...
Comment 15•25 years ago
|
||
Bulk moving [testcase] code to new testcase keyword. Sorry for the spam!
Keywords: testcase
Assignee | ||
Updated•25 years ago
|
Target Milestone: M14 → M15
Updated•25 years ago
|
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
Comment 19•25 years ago
|
||
Heights and widths don't work on table cells, either, when they are less than
the contents' dimensions.
Comment 20•25 years ago
|
||
Comment 21•25 years ago
|
||
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.
Reporter | ||
Comment 22•25 years ago
|
||
PDT- from beta1 removed; nsbeta2 added
Keywords: nsbeta2
Whiteboard: [PDT-]
Comment 23•25 years ago
|
||
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-]
Assignee | ||
Comment 25•25 years ago
|
||
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
Assignee | ||
Comment 26•25 years ago
|
||
Assignee | ||
Comment 27•25 years ago
|
||
Comment 28•25 years ago
|
||
Approving for nsbeta3
Whiteboard: [nsbeta2-] fix-in-hand → [nsbeta2-] [nsbeta3+] fix-in-hand
Updated•25 years ago
|
Priority: P3 → P2
Comment 29•25 years ago
|
||
I will review the patch and if ok, check it in.
Assignee: karnaze → buster
Status: ASSIGNED → NEW
Comment 30•25 years ago
|
||
checking in fix for chris. r=buster
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Whiteboard: [nsbeta2-] [nsbeta3+] fix-in-hand → [nsbeta3+]
Reporter | ||
Comment 31•25 years ago
|
||
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 → ---
Reporter | ||
Comment 32•25 years ago
|
||
Reporter | ||
Comment 33•25 years ago
|
||
Reporter | ||
Comment 34•25 years ago
|
||
Reporter | ||
Comment 35•25 years ago
|
||
Reporter | ||
Comment 36•25 years ago
|
||
Comment 37•25 years ago
|
||
Adding [nsbeta2-] since PR2 is out and [nsbeta3+] is already indicated.
Whiteboard: [nsbeta3+] → [nsbeta3+][nsbeta2-]
Comment 38•25 years ago
|
||
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
Reporter | ||
Comment 39•25 years ago
|
||
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.
Assignee | ||
Comment 40•25 years ago
|
||
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 ago → 25 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 41•25 years ago
|
||
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.
Description
•