Closed
Bug 43745
Opened 25 years ago
Closed 24 years ago
paddingLeft is behaving as same as paddingRight
Categories
(Core :: Layout: Tables, defect, P3)
Tracking
()
People
(Reporter: bora123, Assigned: pierre)
Details
(Keywords: css2, dom2, testcase, Whiteboard: [Hixie-P3])
Attachments
(2 files)
paddingLeft is functioning as expected when it is set thru the object tag. But
its alteration by DOM is functioning like paddingRight. It must me some
misplaced functions in the core code of DOM. Sounds like an easy fix.
<html>
<head>
</head>
<script>
function over() {
var obj=document.getElementById("b1");
obj.setAttribute("style","padding-left:20;");
}
function out() {
var obj=document.getElementById("b1");
obj.setAttribute("style","padding-left:0;");
}
</script>
<body bgcolor="#ababab">
<table cellspacing="0" cellpadding="0" style=" width:100; height:100;">
<tr>
<td align=center style="padding-left:20;" bgcolor="#C0C0C0">TestTest</td>
</tr>
</table>
<br>
<table cellspacing="0" cellpadding="0" style=" width:100; height:100; ">
<tr>
<td onmouseout="out()" onmouseover="over()" id=b1 align=center
bgcolor="#C0C0C0">TestTest</td>
</tr>
</table>
</body>
</html>
|
||
Comment 1•25 years ago
|
||
Pierre, could you have a look at this? I had a quick look and I really don't
think the dom code does anything with those style values except to pass them on
to the style system since the testcase uses setAttribute(), could this be a
style system problem, or table reflow problem?
Assignee: jst → pierre
Status: UNCONFIRMED → NEW
Ever confirmed: true
|
Assignee | |
Comment 2•25 years ago
|
||
I assume you want to move the text within its cell; it means that you shouldn't
apply the style to the cell itself but to the text within the cell. If you apply
the style to the cell, it will move the cell within the table and break the
geometry.
See the testcase that I'm going to attach. It contains examples with tables and
DIVs. Closed as invalid.
Status: NEW → RESOLVED
Closed: 25 years ago
Resolution: --- → INVALID
|
|
Assignee | |
Comment 3•25 years ago
|
||
|
||
Updated•25 years ago
|
Component: DOM Level 2 → DOM Style
|
|
||
Comment 4•25 years ago
|
||
Taking QA Contact on all open or unverified DOM Style bugs...
QA Contact: vidur → ian
|
|
||
Comment 5•24 years ago
|
||
Pierre is wrong. There is a problem here. Like the reporter said around a year
ago, we treat a dynamically added padding-left like a padding-right. I will attach
another testcase. This might be tables related, I couldn't get it to happen with
normal blocks. This is not related to the style attribute, my new testcase doesn't
use one (it dynamically pokes the class).
|
|
||
Comment 6•24 years ago
|
||
this bug has been fixed by Chris's checkin from today morning, Hixie could you
please verify?
|
|
Assignee | |
Comment 8•24 years ago
|
||
Correct, that was fixed by bug 52531 (pull nsTableCellFrame.cpp).
*** This bug has been marked as a duplicate of 52531 ***
Status: REOPENED → RESOLVED
Closed: 25 years ago → 24 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•