Closed
Bug 175070
Opened 22 years ago
Closed 22 years ago
faulty table cell resize after background style attribute change
Categories
(Core :: Layout: Tables, defect, P2)
Tracking
()
RESOLVED
WORKSFORME
Future
People
(Reporter: helmo, Assigned: karnaze)
References
Details
(Keywords: regression, testcase)
Attachments
(3 files, 1 obsolete file)
When using statements like below, the table cell is resized to the minimum
required by the contained text.
document.getElementById('foo').style.background='#fcc';
document.getElementById('foo').width='110';
<table>
<tr><td id=foo>Bar</td></tr>
</table>
Comment 1•22 years ago
|
||
Comment 2•22 years ago
|
||
Browser, not engine ---> DOM Style
The testcase WORKSFORME with Mozilla trunk 20021015xx on WinNT, Linux.
I cannot confirm this bug. The behavior is identical to IE6, as well.
Herman: what build ID of Mozilla are you using? Please include that
in any bug report; thanks -
Assignee: rogerl → jst
Component: JavaScript Engine → DOM Style
QA Contact: pschwartau → ian
Reporter | ||
Comment 3•22 years ago
|
||
After tairing my page appart this is the minimum where is still goes wrong.
Removing the align=right in the inner table tag solves the problem.
The fully implemented page is http://www.wijnstravastgoed.nl/beta
Reporter | ||
Comment 4•22 years ago
|
||
Using:
Mozilla 1.1
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.1) Gecko/20020826
Comment 5•22 years ago
|
||
Confirming bug with Mozilla trunk binary 20021015xx on WinNT.
OS: Linux ---> All.
When I click the button in Herman's testcase, the table remains
the same width in IE6, but collapses in Mozilla to fit the text.
All that the JavaScript is changing is the background color:
<script>
function test()
{
document.getElementById('foo').style.background='#fcc';
}
</script>
<table border=1>
<tr><td>
<table border=1 align=right>
<tr><td id=foo width=600 height=40 class=all>Bar</td></tr>
<tr><td> hhhh</td></tr>
</table>
</td></tr>
</table>
<input type="button" value="set background and width" onClick="test();">
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Linux → All
Comment 6•22 years ago
|
||
This is a table reflow problem.
Assignee: jst → karnaze
Component: DOM Style → HTMLTables
QA Contact: ian → amar
The change of the background color causes a reframe because the
background-attachment for the table cell has the initial value 0 css_unit_null
and it changes to 0 css_enumerated. This throws in a new anonymous colframe
which blows up the table width calculations.
Depends on: 171830
Comment 9•22 years ago
|
||
regression between linux trunk builds 2002050902 and 2002051010
potential candidates are bug 120107 and bug 141021
Keywords: regression,
testcase
Comment 10•22 years ago
|
||
while attachment 107246 [details] can be fixed by the style system bug 171830 this
testcase causes a reframe inside the style system. Due to the reframe a
colframe will be initialized,( placing a break point at
http://lxr.mozilla.org/seamonkey/search?string=nsTableColFrame%3A%3AResetSizing
shows that perfectly ) There may be several possibilities to fix the bug a) fix
the reframe which seems to me the source b) fix the reflow as it should be
able to handle this.
Updated•22 years ago
|
Priority: -- → P2
Updated•22 years ago
|
Keywords: mozilla1.3
Comment 12•22 years ago
|
||
This should now be fixed... please retest (modulo that last testcase; can't tell
what that's testing).
Comment 13•22 years ago
|
||
Comment on attachment 107280 [details]
testcase with background-attach
2nd and 3rd testcase (attachment 103241 [details] and attachment 107246 [details]) still
show problems on 2003030609 win build. 4th testcase (attachment 107280 [details]
is worksforme)
Attachment #107280 -
Attachment is obsolete: true
Updated•22 years ago
|
Summary: faulty cell resize after attribute change → faulty table cell resize after background style attribute change
Comment 14•22 years ago
|
||
all 4 testcases workforme with linux trunk 20030307 (gotta test with a build
after 20030306)
Comment 15•22 years ago
|
||
all testcases worksforme moz 2003031408/win98
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•