343356 - Can't scroll to view left portion of this page (RTL table with align="center" overflows to the left of its LTR container)

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect)

Description

[Niro]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060627 Minefield/3.0a1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20060627 Minefield/3.0a1

Go to the URL and notice that there is more content in the left area but actually you can not see it, nor scroll the page.

Reproducible: Always

Comment 1

[Uri Bernstein]

The issue here seems to be as follows:
- The page itself is LTR.
- It contains an RTL table with align="center".
- The table contains long unbreakable text, causing it to overflow.

The problem is that the center-aligned, RTL, table is overflowing to the left of the LTR body. Since the policy is not to allow scrolling to the left of the origin (in LTR pages), we don't allow scrolling to the overflowing part of the table.

The RTL table should really overflow to the right, as it indeed does if align="center" is not specified.

I'll attach a testcase immediately.

Comment 2

[Uri Bernstein]

Attachment: testcase

This testcase uses a fixed-width LTR div (with blue border), instead of the body, as the container for the RTL table, so that it won't depend on the window width.

Comment 3

[Uri Bernstein]

The proposed fix for bug 328181 *should* have fixed this bug as well (when aligning a centered over-constrained frame, align it based on the direction of its containing block, not the direction of the frame itself).

The problem is that in case of a table frame, the container is actually an nsOuterTableFrame, which has the same direction as the table frame. So in this case, I think we need to look at the grandparent's mDirection, rather than the parent's.

So for now I'm just marking this as depending on bug 328181. We can try fixing this as part of fixing that bug, or we can come back to this after fixing that one.

Comment 4

[Uri Bernstein]

This would be fixed by the latest patch (v3) on bug 328181. It wasn't fixed with previous versions of that patch because I was using the parent's direction, rather than the containing block's.

Comment 5

[Uri Bernstein]

Fixed by the fix for bug 328181.

Tested with:
Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.9a1) Gecko/20061214 Minefield/3.0a1

Comment 6

[(no longer active)]

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