1337425 - [RTL] Highlights introduction in Activity Stream lacks RTL

Status: RESOLVED DUPLICATE of bug 1388377

(Firefox for Android Graveyard :: Theme and Visual Design, defect, P2)

Description

[Itiel]

Attachment: Highlights introduction

STR:
1. Install Switchboard Experiments add-on on a FRESH installed Nightly
2. In about:experiments, tap on Activity stream
3. Settings -> Advanced -> Enable Activity stream
4. Open a New Tab
5. Observe the Highlights introduction.

AR:
It's not RTL-friendly.

ER:
a. The little fox and the globe image should be aligned to the left
b. The DISMISS button should be aligned to the right
c. The Highlights introduction should be aligned to the right

See attached screenshot (the introduction isn't shown there, it's probably due to a translation issue).

Comment 1

[Tomer Cohen :tomer]

Attachment: Screenshot of Nightly new tab

This bug reproduces on plain install of Nightly, and soon on every installation. Any chance to have it fixed? Changing the alignment to 'end' instead of 'right' could be a one-line change. 

UPDATED steps to reproduce:
a. Prerequisite: RTL locale of Android to an RTL locale or enable Android's Force RTL layouts on Settings → Developers.
b. Install Nightly or delete application data to have fresh start. 
c. Open new tab.

Comment 2

[Tomer Cohen :tomer]

(In reply to Tomer Cohen :tomer from comment #1)
> UPDATED steps to reproduce:
> a. Prerequisite: RTL locale of Android to an RTL locale or enable Android's
> Force RTL layouts on Settings → Developers.

Alternative steps to reproduce:
Instead of having RTL locale on Android, you can also change the application language directly, but keep in mind that the profile should be clean.

Comment 3

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

bug 1388377 will change this UI around - we should do/verify the RTL afterwards.

Comment 4

[Chenxia Liu [:liuche]]

This is for the empty state, and should only be shown once, so dropping this to P2 rank 3. (And it should probably just work)

Comment 5

[Michael Comella (:mcomella) [NI reported issues only: ex-Mozilla]]

This should just work after bug 1388377: it's only a TextView which should just work for RTL.