1159478 - [RTL][Browser] Back And Forward buttons are reversed in RTL languages

Status: RESOLVED INVALID

(Firefox OS Graveyard :: Gaia::Browser, defect, P1)

Description

[Derek Harris [:DerekH] - Dont need info. Last day 6/5/15]

Attachment: Browser navigation RTL Logcat

Description:
The back and forward buttons will display as flipped in the browser when the user is in an RTL language such as Arabic


Repro Steps:
1) Update a Flame to 20150428010206
2) Set language to Arabic (or another RTL language)
3) Open Browser app
4) Navigate to 2 or more sites
5) Observe Back and Forward buttons


Actual:
Back and Forward buttons are flipped in RTL


Expected:
Back and Forward buttons remain the same in RTL as they are in LTR layouts

Environmental Variables:
Device: Flame 3.0 (319mb)(Kitkat)(Full Flash)
Build ID: 20150428010206
Gaia: 0636405f0844bf32451a375b2d61a2b16fe33348
Gecko: caf25344f73e
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0


Repro frequency: 13/13
See attached: Logcat, Video - https://www.youtube.com/watch?v=giyiEK_KlHM&feature=youtu.be

Comment 1

[Derek Harris [:DerekH] - Dont need info. Last day 6/5/15]

This issue DOES occur on Flame 2.2

Back and Forward buttons remain the same in RTL as they are in LTR layouts

Environmental Variables:
Device: Flame 2.2 (319mb)(Kitkat)(Full Flash)
Build ID: 20150428002500
Gaia: 9f6b1b9082662ba2c14168fc66bb02b4df3141e5
Gecko: e79c19bf19bf
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

===========================================================================================================

RTL is not supported on devices earlier than 2.2, so no branch checks were done for these devices

Comment 2

[Delphine Lebédel [:delphine - Use needinfo]]

I had understood that Browser buttons are supposed to remain the same as in LTR, unless I'm missing something. Is this a regression? 
Triage: P1

Comment 3

[Derek Harris [:DerekH] - Dont need info. Last day 6/5/15]

Yes from my understanding the Browser buttons are supposed to remain the same as LTR.

As of right now this issue is occuring on Flame 3.0, and 2.2, but if I remember correctly, the browser buttons use to be the same as the LTR layout, so in that regaurd yes this is a regression

Comment 4

[Delphine Lebédel [:delphine - Use needinfo]]

Ok nominating to block on this then

Comment 5

[Delphine Lebédel [:delphine - Use needinfo]]

Adding the regression keyword, please take out if unnecessary

Comment 6

[Delphine Lebédel [:delphine - Use needinfo]]

Spoke with Ahmed offline, he confirmed him and Stephany had agreed this should look this way. For some reason thought the behavior should remain the same as in LTR for legacy reasons. Closing as resolved invalid