318802 - Table cells overlap if one is DIR=rtl and contains a DIV with padding

Status: RESOLVED DUPLICATE of bug 258928

(Core :: Layout: Tables, defect)

Description

[Ilan Tayary]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.0; .NET CLR 1.1.4322)
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

In the example, it seems that the padding is done to the wrong side. instead of padding, e.g. pushing the left-margin of the contents of the DIV to the right, the renderer pushes it to the left, thus overlapping the contents of the left cell.

Note that the "width: 100%" is critical for the error, as well as the "DIR=rtl" and the "padding: ..."

This problem used to happen in Firefox 1.0.x, and still remains in 1.5

Reproducible: Always

Steps to Reproduce:
1. open a file that contains the following HTML:

<table width=400><tr><td>TEXT TEXT TEXT</td>
<td dir=rtl><div style="width: 100%; padding:0px 120px 0px 0px;">TEXT TEXT TEXT TEXT TEXT TEXT </div></td></tr></table>


Actual Results:  
The right-hand-side (2nd) cell overlaps the left (1st) cell

Expected Results:  
The cell contents should have a space between them because of the padding of the DIV

Comment 1

[Reed Loden [:reed]]

Confirming on Firefox 1.5 [Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5 ID:2005111116].

Comment 2

[logan]

Attachment: testcase

Comment 3

[logan]

Attachment: screenshot

Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8) Gecko/20051111 Firefox/1.5

Comment 4

[Bernd]

Attachment: more visual testcase

Comment 5

[Bernd]

Reed, there are several issues with your confirming of the bug.
1.) why did you remove the Trunk from the Version field, did you check?
    This is also on trunk.
2.) Did you read  http://groups.google.de/group/netscape.public.mozilla.layout/browse_thread/thread/4b44ff47ef062b20/d30ce1ee477eceb0?lnk=st
3.) Are you sure that this a table bug and not bug in block and line layout?

Comment 6

[Bernd]

In other words searching for rtl and padding brings up bug 258928, I guess this a dupe of it.

Comment 7

[Reed Loden [:reed]]

(In reply to comment #5)
> Reed, there are several issues with your confirming of the bug.
> 1.) why did you remove the Trunk from the Version field, did you check?
>     This is also on trunk.

Yes, I know. However, it's also on the branch. Most bugs on the branch are also on the trunk, not the other way around. Frankly, it should probably be "unspecified" to mean both branch and trunk.

> 2.) Did you read 
> http://groups.google.de/group/netscape.public.mozilla.layout/browse_thread/thread/4b44ff47ef062b20/d30ce1ee477eceb0?lnk=st

No, I've never seen that post before. I'll read it fully later and keep it in mind for future bugs. Thanks.

> 3.) Are you sure that this a table bug and not bug in block and line layout?

This bug originally came from a discussion on IRC, which I was just confirming that I see the bug in Gecko 1.8. I hadn't checked for dupes yet, considering logan was the main one helping the reporter out.

(In reply to comment #6)
> In other words searching for rtl and padding brings up bug 258928, I guess this
> a dupe of it.

Thanks again; marking it as such.

*** This bug has been marked as a duplicate of 258928 ***

Comment 8

[(no longer active)]

Mass-assigning the new rtl keyword to RTL-related (see bug 349193).