526477 - [RTL] rtl pages clipped on left

Status: RESOLVED WORKSFORME

(Firefox for Android Graveyard :: General, defect)

Description

[Tuukka Tolvanen (sp3000)]

Attachment: testcase

On rtl pages, the overflow is cropped out on the left side.

Examples:
http://ar-ar.facebook.com/ (may need to manually select non-mobile site and then arabic at bottom)
http://likud.org.il/
http://fa.wikipedia.org/

The width of the canvas appears to be correct, but the content is set in it such that the left edge is clipped and there's empty space on the right.

Comment 1

[Tuukka Tolvanen (sp3000)]

Attachment: ar fb screenshot

Comment 2

[Tuukka Tolvanen (sp3000)]

This is fennec1.0b4; I think b3 behaved differently on some of these cases.

Comment 3

[Juha Seppänen]

Attachment: Same test page on N900 running Beta 5

Same thing on Beta 5

Comment 4

[Oleg Romashin (:romaxa)]

I'm not sure but it seems that IgnoreScrollFrame rendering way and rootScrollFrame scroll position conflicting between each other.

LTR fb we are calculating scrollPos from left/top page point (0,0), in RTL fb we  are calculating scrollPos from scrollMaxX, 0  page point in layout coordinates.

And to be able render left part of RTL fb in IgnoreScrollFrame mode, we should give negative layout coordinates...

roc, any ideas why here is the such strange behavior?

Comment 5

[Oleg Romashin (:romaxa)]

If on RTL page we call:
javascript:window.scrollX; 

it show negative value... but
javascript:window.scrollMaxX;
show positive value..
is it by standard or or what?

Btw: in google-chrome RTL facebook also broken.

Comment 6

[Robert O'Callahan (:roc) (email my personal email if necessary)]

(In reply to comment #4)
> And to be able render left part of RTL fb in IgnoreScrollFrame mode, we should
> give negative layout coordinates...

That sounds right. Does it work?

> roc, any ideas why here is the such strange behavior?

That's just what we (and other browsers) do.

Comment 7

[Aakash Desai [:aakashd]]

Fennec needs to support rtl in general. THere's a bug on file for this and marking this bug as dependent on the other.

Comment 9

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

Litmus Test case : https://litmus.mozilla.org/show_test.cgi?searchType=by_id&id=12774 added.

Disabled until this bug is fixed.

Comment 10

[Mark Finkle (:mfinkle) (use needinfo?)]

Naoki - I assume this bug is still a problem?