Closed
Bug 5798
Opened 26 years ago
Closed 26 years ago
table height=% should ignore body margins
Categories
(Core :: Layout: Tables, defect, P3)
Core
Layout: Tables
Tracking
()
VERIFIED
FIXED
M11
People
(Reporter: hyp-x, Assigned: karnaze)
References
Details
(Whiteboard: [TESTCASE])
Attachments
(1 file)
|
(deleted),
text/html
|
Details |
The table in the following example is just too high to fit
without scrolling regardless of the window size.
The problem is in gecko 100% means the window height,
while in other browsers (ie4, ns4.5) it is the window height
minus the margins (2*marginheight) and table border.
Checked M4 and 1999042908
---------
<html>
<body>
<table border=5 height=100%>
<tr>
<td>
This table is set to a height of 100%.<br>
It should just fit the page.
</table>
</body>
</html>
|
Assignee | |
Updated•26 years ago
|
Status: NEW → ASSIGNED
Target Milestone: M7
|
|
Assignee | |
Comment 1•26 years ago
|
||
Moving to M9.
|
Reporter | |
Updated•26 years ago
|
Summary: table heigth=% should ignore margins and border → table heigth=% should ignore body margins
Whiteboard: [TESTCASE]
|
|
Reporter | |
Comment 2•26 years ago
|
||
I cannot simplify the testcase any more, I just made it a little bit cleaner.
Adding as attachment for easier viewing from the browser.
I just discovered that the table fits when the body has marginheight=0
So I'm changing the summary a little bit.
|
|
Reporter | |
Comment 3•26 years ago
|
||
|
||
Updated•26 years ago
|
OS: Windows 98 → All
Hardware: PC → All
Summary: table heigth=% should ignore body margins → table height=% should ignore body margins
|
|
||
Comment 5•26 years ago
|
||
The test page from bug 7158 is available at:
http://www.bath.ac.uk/%7Epy8ieh/internet/projects/mozilla/mondrian8.html
This is a very good test case.
It seems that Mozilla is calculating the table's height as a percentage of the
height of the container including any padding, margins, and border. The height
should only be taking the inner content height into account.
|
|
Assignee | |
Updated•26 years ago
|
Target Milestone: M9 → M11
|
|
Assignee | |
Comment 6•26 years ago
|
||
Moving to M11.
|
|
Assignee | |
Updated•26 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 26 years ago
Resolution: --- → FIXED
|
|
Assignee | |
Comment 7•26 years ago
|
||
Fixed with latest checkin.
|
||
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
|
|
||
Comment 8•25 years ago
|
||
Using 9/16 Apprunner, verified fixed.
You need to log in
before you can comment on or make changes to this bug.
Description
•