731594 - [bidi] Reversed text on Facebook photo galleries

Status: RESOLVED FIXED

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

Description

[Tomer Cohen :tomer]

Attachment: screenshot

My sister was able to reproduce something very similar to bug 704837 and bug 730671, but this time on Facebook photo galleries. 

Steps to reproduce:
a. Enter a facebook page (For example PizzaHut's https://www.facebook.com/PizzaHut.Israel)
b. Click on a photo published on their wall.
c. You'll get redirected to the photo gallery with a gray background. 
d. Navigate to next/previous photo, and watch the photo description at the right side appearing reversed. 

Note: We can't place direct link to a photo, as this won't trigger the gallery viewer and instead will show the photo page (white background).


Tested with Firefox 10.0.2 on Windows 7, and Aurora Mozilla/5.0 (X11; Linux i686; rv:12.0a2) Gecko/20120228 Firefox/12.0a2. Reproduces on other Facebook pages as well. 


Please set as duplicate if you think the same issue described on one of the other bugs.

Comment 1

[Jonathan Kew [:jfkthame]]

Could you please test with Aurora (FF 12)? I think this is probably another example of bug 718236, in which case it should already be fixed on Aurora (and Nightly), and will be fixed in the next Beta release of FF 11.

Comment 2

[Jonathan Kew [:jfkthame]]

(In reply to Jonathan Kew (:jfkthame) from comment #1)
> Could you please test with Aurora (FF 12)? I think this is probably another
> example of bug 718236, in which case it should already be fixed on Aurora
> (and Nightly), and will be fixed in the next Beta release of FF 11.

Sorry, just re-read and realized that you said it occurred on Aurora as well.

How about Nightly - is the problem present there too?

Comment 3

[Jonathan Kew [:jfkthame]]

OK, this is very weird. I was about to comment that I couldn't reproduce the issue as shown in your screenshot - and then it happened to me. It seems to depend on whether I arrive at that picture by stepping forward or backward through the gallery. If I reach it from the _preceding_ picture, by clicking the Next (rightward) arrow, the text displays correctly; but if I go beyond it and then return there by clicking the Previous (leftward) arrow, I see the reversed text as in your screenshot.

Similar reversed text appears in some other picture comments as well, and in some cases is dependent on how I approach the picture within the set.

(Confirmed with FF 10, Aurora 12 and Nightly builds, although the exact examples that display wrong seem to vary a bit. In one case, I even saw a line of text appear correctly at first and then flip around after a fraction of a second, though I was not able to reproduce this behavior consistently.)

Comment 4

[Tomer Cohen :tomer]

We are aware of user complains about this issue. I'd suggest prioritizing it. 

http://mozilla.org.il/board/viewtopic.php?f=9&t=10982
https://www.facebook.com/MozillaIsrael/posts/360399213991581
http://input.mozilla.com/he/opinion/2766178

Comment 5

[Tomer Cohen :tomer]

Any chance to resolve this issue any time soon? It make it very difficult to view photo galleries using Firefox, and it reproduce every time.

If this bug is caused by non-standard code on the Facebook side, it might be good idea to ask them to fix it.

Comment 6

[(no longer active)]

I'm bisecting to figure out exactly what regressed this.

Comment 7

[(no longer active)]

The bisection is now finished:

The first bad revision is:
changeset:   79883:00b0ddf1ab96
user:        Simon Montagu <[e-mail]>
date:        Mon Nov 07 16:26:50 2011 +0200
summary:     Test the containing frame for continuations, not the child. Bug 698706, r=roc

Comment 8

[Simon Montagu :smontagu]

Attachment: w-i-p

I'm going to dogfood with this for a while before asking review. Tryserver builds will be available at http://ftp.mozilla.org/pub/mozilla.org/firefox/try-builds/[e-mail]-b621fca09303 when ready.

Comment 9

[Tomer Cohen :tomer]

I think that the tryserver build is fixing well these issues.

Comment 10

[Tomer Cohen :tomer]

I am on Aurora (14) and this issue still occur. Please land this fix as soon as possible as this issue affects people in the stable channel and I am hearing complains every day about Firefox lacking support in Chrome and people that wish to downgrade only because of this issue.

Is it possible to land a fix to the beta channel so people will get the fix in less than six weeks from now?

Comment 11

[(no longer active)]

As far as I can tell, we still don't have a patch here which is ready for review...

Comment 12

[Simon Montagu :smontagu]

Comment on attachment 610954 [details] [diff] [review]
w-i-p

I haven't found any regressions from this.

The patch catches some more cases where we had a dangling bidi-isolated sub-paragraph which wasn't going through bidi resolution at all. It does this by checking at the end of TraverseFrames and also when creating a new sub-paragraph whether there is any text in the current subparagraph

Comment 13

[Simon Montagu :smontagu]

https://hg.mozilla.org/integration/mozilla-inbound/rev/cfa3d6007436
Unfortunately with no testcase. I spent some time trying to come up with a minimized testcase for this but didn't get anywhere.

Comment 14

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/cfa3d6007436

Comment 17

[Guy]

is it possible to get a version to test?

Comment 18

[Simon Montagu :smontagu]

(In reply to Guy from comment #17)
> is it possible to get a version to test?

http://www.mozilla.org/beta/

Comment 19

[Guy]

Hi Simon,

When there is a number in the middle of the sentence, the sentence gets out of order (the words after the number gets rearranged to the beginning of the sentence).

This one occurred to me also in Facebook.com, when replying on an Event's wall.

This is what it should look like:
 תיק נוח שאפשר לשים בו 3 ליטר מים לאדם
This is the result:
ליטר מים לאדםתיק נוח שאפשר לשים בו 3

Comment 20

[Simon Montagu :smontagu]

Guy, can you open a new bug report about that, with clear instructions how to reproduce the bug? It's not the same issue as this one.