Closed Bug 1337407 Opened 7 years ago Closed 7 years ago

[RTL] Separators in Activity Stream items' context menu should be aligned to the left

Categories

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

defect

Tracking

(firefox56 fixed)

VERIFIED FIXED
Firefox 56
Tracking Status
firefox56 --- fixed

People

(Reporter: itiel_yn8, Assigned: liuche)

References

Details

(Keywords: rtl, Whiteboard: [MobileAS] 1.26)

Attachments

(3 files)

STR:
1. Install Switchboard Experiments add-on on the latest Nightly
2. In about:experiments, tap on Activity stream
3. Settings -> Advanced -> Enable Activity stream
4. Open a New Tab
5. Click on the 3 dots of one of the suggested items
6. Observe the separators

AR:
The separators are aligned to the right.

ER:
They should be aligned to the left.

See attached screenshot.
Summary: [RTL] Separators in Activity Stream items' context menu should be align to the left → [RTL] Separators in Activity Stream items' context menu should be aligned to the left
I am not sure if URLs should be right-aligned. I'd prefer having the URL as well as the page title displayed according to their language, as in dir=auto.
(In reply to Tomer Cohen :tomer from comment #1)
> I am not sure if URLs should be right-aligned. I'd prefer having the URL as
> well as the page title displayed according to their language, as in dir=auto.

I was talking about the separators, not URLs :)
Sorry. Now that I see it, I am sure you're right.
Priority: -- → P2
Whiteboard: [MobileAS]
Rank: 1
Whiteboard: [MobileAS] → [MobileAS] 1.26
Assignee: nobody → liuche
Added a rotated drawable in the ldrtl (layout direction right to left) dir.
Comment on attachment 8888607 [details]
Bug 1337407 - Add rtl divider resource.

https://reviewboard.mozilla.org/r/159610/#review165352

::: mobile/android/app/src/main/res/drawable-ldltr/as_contextmenu_divider.xml:1
(Diff revision 1)
> +../drawable/as_rtl_contextmenu_divider.xml

nit: that file should be called `as_ltr_contextmenu_divider` because it's the ltr configuration.

That being said, I'd actually prefer `as_contextmenu_divider_helper` or similar, because it's otherwise unclear that `as_contextmenu_divider` is the one that's intended to be used and `as_contextmenu_divider_helper` only exists because it's impossible to define a bidirectional drawable without it. Maybe add a comment to `as_contextmenu_divider_helper` to that effect too.
Attachment #8888607 - Flags: review?(michael.l.comella) → review+
Iteration: --- → 1.26
Priority: P2 → P1
Pushed by cliu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/080aa07d32df
Add rtl divider resource. r=mcomella
https://hg.mozilla.org/mozilla-central/rev/080aa07d32df
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 56
LGTM.
Status: RESOLVED → VERIFIED
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.