Closed
Bug 1232054
Opened 9 years ago
Closed 9 years ago
[TV] Marketplace UI should support RTL
Categories
(Firefox OS Graveyard :: Gaia::TV, defect, P4)
Tracking
(feature-b2g:2.6?)
RESOLVED
FIXED
feature-b2g | 2.6? |
People
(Reporter: ddurst, Assigned: jyeh)
References
Details
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
I don't know if this is addressed already or not, so I'm filing to ensure that it's covered. (I *think* I saw this in the PR conversations, but just in case...)
Marketplace UI should support RTL languages at launch.
Comment 1•9 years ago
|
||
Update partner RTL support in 2.1:
"
Basically, we use only LTR direction for each languages.
UI lauyout is broken when we tried to use RTL(attached page.1).
so we disbaled the RTL direction of l10n for v2.1[1].
For arabic language, icu renders from right to left.
icu decides the direction for writing based on character code.
(please refer attached page.2)
[1]: https://github.com/mozilla/stingray-gaia/blob/master/shared/js/l10n.js#L1470
"
Comment 2•9 years ago
|
||
International Components for Unicode: http://site.icu-project.org/
Updated•9 years ago
|
Priority: -- → P3
Updated•9 years ago
|
feature-b2g: --- → 2.6?
Reporter | ||
Updated•9 years ago
|
Priority: P3 → P4
Comment 3•9 years ago
|
||
Sounds like we descoped this feature on 2.5. Should we move it from the meta bug 1204762?
Flags: needinfo?(jocheng)
Summary: [TV][2.5] Marketplace UI should support RTL → [TV] Marketplace UI should support RTL
Updated•9 years ago
|
feature-b2g: 2.6? → ---
Flags: needinfo?(jocheng)
Updated•9 years ago
|
feature-b2g: --- → 2.6?
Comment 4•9 years ago
|
||
(In reply to Evelyn Hung [:evelyn] from comment #3)
> Sounds like we descoped this feature on 2.5. Should we move it from the meta
> bug 1204762?
Removed
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → jyeh
Assignee | ||
Comment 5•9 years ago
|
||
Assignee | ||
Comment 6•9 years ago
|
||
Landed in marketplace-tv-front-end master.
https://github.com/mozilla-b2g/marketplace-tv-front-end/commit/b8962ec527b700514f2885ed9c3b1d02c8a41fa1
The dev server did not update accordingly, so leave this bug opened until it updates.
Reporter | ||
Comment 7•9 years ago
|
||
Closing because this should be landed now.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•