Closed Bug 652259 Opened 9 years ago Closed 9 years ago

[RTL] New Theme Style shows arrow in wrong place for opening a new tab

Categories

(Firefox for Android Graveyard :: General, defect, P2)

ARM
Android
defect

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: nhirata, Assigned: linostar)

Details

Attachments

(2 files)

Mozilla/5.0 (Android; Linux armv71; rv6.0a1) Gecko/20110422 Firefox/6.0a1 Fennec/6.0a1
Device: Thunderbolt
OS: Android 2.2

1. install the arabic addon from :http://ftp.mozilla.org/pub/mozilla.org/mobile/nightly/latest-mozilla-central-linux-l10n/
2. set the language to arabic
3. go to www.google.com
4. open a new tab by long tapping and clicking in the first link

Expected: Arrow will be in the right order from the dialog
Actual: the Arrow is in the wrong position

Note:
1. based on testing from Bug 618989
Assignee: nobody → linux.anas
Status: NEW → ASSIGNED
Attached patch patch v1Splinter Review
Attachment #528806 - Flags: review?(21)
Summary: RTL: New Theme Style shows arrow in wrong place for opening a new tab → [RTL] New Theme Style shows arrow in wrong place for opening a new tab
(In reply to comment #2)
> Created attachment 528806 [details] [diff] [review] [review]
> patch v1

Can you check that your patch does not regress bug 605930?
(In reply to comment #3)
> Can you check that your patch does not regress bug 605930?

Tested. No regression.
Comment on attachment 528806 [details] [diff] [review]
patch v1

I will change the code I've indicated in the previous comment to use "normal" instead of "ltr" for parity.
Verified fixed on build: Mozilla /5.0 (Android;Linux armv7l;rv:6.0a1) Gecko/20110510 Firefox/6.0a1 Fennec/6.0a1 
Device: LG Optimus 2X (Android 2.2)
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.