Closed
Bug 1218733
Opened 9 years ago
Closed 9 years ago
[RTL][Settings]"WAP" string should locates at right side of Arabic "push" string.
Categories
(Mozilla Localizations :: ar / Arabic, defect, P3)
Tracking
(b2g-master affected)
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
b2g-master | --- | affected |
People
(Reporter: xiongfuchao, Unassigned)
References
Details
(Whiteboard: [2.5-rtl-test-run])
Attachments
(1 file)
(deleted),
image/png
|
Details |
[1.Description]:
[RTL][Arieskk_v2.5][Flamekk_v2.5][Settings]Set your phone language to Arabic, then operate to the message setting page, you can find "WAP" string is on right side of Arabic "push" string in subheader but locates at the left in tab list.
See attachment:Screenshot_WAP.png
[2.Testing Steps]:
Precondition: Insert SIM cards into device and Set your phone language to Arabic.
1. Launch Settings app.
2. Tap "Messaging Settings".
3. Check strings at this page.
[3.Expected Result]:
3. "WAP" string should be at the right side of arabic "push" string.
[4.Actual Result]:
3. "WAP" string locates at the left side of Arabic "push" string.
[5.Reproduction build]:
Device: AriesKK_2.5 [Affected]
Build ID 20151026223015
Gaia Revision b564b21e08ffc3e4962f08850843c7482932ee7b
Gaia Date 2015-10-26 14:22:21
Gecko Revision https://hg.mozilla.org/mozilla-central/rev/6c7c983bce46a460c2766fbdd73283f6d2b03a69
Gecko Version 44.0a1
Device Name aries
Firmware(Release) 4.4.2
Firmware(Incremental) eng.worker.20151026.214940
Firmware Date Mon Oct 26 21:49:48 UTC 2015
Bootloader s1
Device: FlameKK_v2.5[Affected]
Build ID 20151026030217
Gaia Revision a677ddd3aa3a81058775938bd56008d96dbc78b0
Gaia Date 2015-10-26 04:48:31
Gecko Revision https://hg.mozilla.org/mozilla-central/rev/5ca03a00d26823ce91ee0eaa2937bed605bd53c1
Gecko Version 44.0a1
Device Name flame
Firmware(Release) 4.4.2
Firmware(Incremental) eng.cltbld.20151026.070911
Firmware Date Mon Oct 26 07:09:23 EDT 2015
Firmware Version v18D v4
Bootloader L1TC000118D0
[6.Reproduction Frequency]:
occasionally Recurrence,5/5
[7.TCID]:
15994
Reporter | ||
Updated•9 years ago
|
QA Whiteboard: [rtl-impact]
status-b2g-master:
--- → affected
Updated•9 years ago
|
Priority: -- → P3
Updated•9 years ago
|
Flags: needinfo?(nefzaoui)
Comment 1•9 years ago
|
||
It's a loca issue: RLM must be added in front of the string.
Comment 2•9 years ago
|
||
Hi Delphine,
Could you help with this l10n bug? Thanks!
Flags: needinfo?(lebedel.delphine)
Updated•9 years ago
|
Component: Gaia::Settings → Gaia::L10n
Comment 3•9 years ago
|
||
(In reply to Augustin Trancart [:autra] from comment #1)
> It's a loca issue: RLM must be added in front of the string.
I would prefer to aim at fixing it globally, instead of for a single string. It seems like a checkbox widget can just hold its label in FSI/PDI?
Comment 4•9 years ago
|
||
Taking out the ni since this is going to be fixed globally
Flags: needinfo?(lebedel.delphine)
Comment 5•9 years ago
|
||
Let's temporarily move this bug to “Product: Mozilla Localizations” and double check if it happens on the other RTL languages.
Component: Gaia::L10n → ar / Arabic
Product: Firefox OS → Mozilla Localizations
Comment 6•9 years ago
|
||
(In reply to Zibi Braniecki [:gandalf][:zibi] from comment #3)
> (In reply to Augustin Trancart [:autra] from comment #1)
> > It's a loca issue: RLM must be added in front of the string.
>
> I would prefer to aim at fixing it globally, instead of for a single string.
> It seems like a checkbox widget can just hold its label in FSI/PDI?
That's precisely what will break the string. If we isolate it with FSI/PDI, the calculated direction would be LTR, and WAP would be at the left.
We should let this string without any markers, except if it is to be used in a dir="auto" context, or inside another l10n string (as it would be enclosed in LRI/PDI I guess), and in this case, we must somehow add a RLM in front of it, either by localizers or in l10n.js.
Btw, I cannot reproduce it with today's master, either in arabic, mirrored english, so it should have been fixed somehow.
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(nefzaoui)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•