Closed
Bug 1337425
Opened 8 years ago
Closed 7 years ago
[RTL] Highlights introduction in Activity Stream lacks RTL
Categories
(Firefox for Android Graveyard :: Theme and Visual Design, defect, P2)
Firefox for Android Graveyard
Theme and Visual Design
Unspecified
Android
Tracking
(firefox53 unaffected, firefox54 unaffected, firefox55 affected)
RESOLVED
DUPLICATE
of bug 1388377
Tracking | Status | |
---|---|---|
firefox53 | --- | unaffected |
firefox54 | --- | unaffected |
firefox55 | --- | affected |
People
(Reporter: itiel_yn8, Unassigned)
References
Details
(Keywords: rtl, Whiteboard: [MobileAS] 1.27)
Attachments
(2 files)
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•7 years ago
|
||
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.
Updated•7 years ago
|
status-firefox53:
--- → unaffected
status-firefox54:
--- → unaffected
status-firefox55:
--- → affected
OS: Unspecified → Android
Comment 2•7 years ago
|
||
(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.
Updated•7 years ago
|
Priority: -- → P2
Whiteboard: [MobileAS]
Updated•7 years ago
|
Iteration: --- → 1.27
Priority: P2 → P1
Whiteboard: [MobileAS] → [MobileAS] 1.27
Comment 3•7 years ago
|
||
bug 1388377 will change this UI around - we should do/verify the RTL afterwards.
Depends on: 1388377
Updated•7 years ago
|
Iteration: 1.27 → 1.28
Updated•7 years ago
|
Iteration: 1.28 → 1.29
Updated•7 years ago
|
Rank: 3
Priority: P1 → P2
Comment 4•7 years ago
|
||
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)
Updated•7 years ago
|
Iteration: 1.29 → ---
Comment 5•7 years ago
|
||
This should just work after bug 1388377: it's only a TextView which should just work for RTL.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → DUPLICATE
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•