604540 - [RTL] The update button overlaps the LTR Fennec title/logo in about: fennec when in RTL

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: General, defect)

Description

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

Attachment: Maemo 5 GTK screenshot

Mozilla/5.0 (Maemo;Linux armv71; rv:2.0b8pre) Gecko/20101014 Firefox/4.0b8pre Fennec/4.0b2pre
Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101014 Firefox/4.0b8pre Fennec/4.0b2pre

1. place the browser in RTL format : ( ie download/install the plugin from :
http://people.mozilla.com/~nhirata/html_tp/installplugin.html 
and set the language to arabic)
2. go to about:fennec
3. look at the top.

Expected:  The button will be to the left, the logo is RTL
Actual: 
1. the button overlays on top of the fennec wording and logo.
2. the Fennec logo and title is in LTR.

Note:
1. Not sure if Maemo should also have this button?  If so, should it work for Maemo?

Comment 1

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

Attachment: Android Droid 2 Screenshot

Comment 2

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Comment 3

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

Comment on attachment 483975 [details] [diff] [review]
Patch

Wrong patch?

Comment 4

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

Attachment: Patch

Comment 5

[Vivien Nicolas (:vingtetun) (:21) - (NOT reading bugmails, needinfo? please)]

http://hg.mozilla.org/mobile-browser/rev/30f841af19c0

Comment 6

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

verified:
Mozilla/5.0 (Maemo;Linux armv71; rv:2.0b8pre) Gecko/20101019 Firefox/4.0b8pre Fennec/4.0b2pre
Mozilla/5.0 (Android; Linux armv71; rv2.0b8pre) Gecko/20101019 Firefox/4.0b8pre Fennec/4.0b2pre