Closed
Bug 1139209
Opened 10 years ago
Closed 9 years ago
[RTL][System] Parenthesis is mis-placed on notification and dialogs when Marketplace app contains parenthesis in the title
Categories
(Mozilla Localizations :: ar / Arabic, defect)
Tracking
(b2g-v2.2 verified, b2g-master verified)
VERIFIED
FIXED
People
(Reporter: ychung, Unassigned)
Details
(Whiteboard: [3.0-Daily-Testing][systemsfe] LocRun2.2)
Attachments
(3 files)
Description:
When a Marketplace app includes parenthesis at the end in the title, the related messages incorrectly display the parenthesis.
Repro Steps:
1) Update a Flame to 20150303010233.
2) Set the device language in Arabic under Settings > Language.
3) Go to Marketplace, and search the app "Aeropuerto El Dorado".
4) Select "Free".
5) Observe the closing parenthesis on the install dialog.
6) Select "Install".
7) When the app is installed, observe closing parenthesis on the notification at the bottom.
8) Go to the homescreen,and enter the edit mode by long-pressing any app.
9) Select the "x" button on the installed app.
10) Observe the closing parenthesis on the delete dialog.
Actual:
5, 7, 10) The closing parenthesis is not placed at the end of "oficial"
Expected:
5, 7, 10) Parentheses are placed correctly.
Environmental Variables:
Device: Flame 3.0 (KK, 319mb, full flash)
Build ID: 20150303010233
Gaia: c8ed1085a67490a1ecd7f275e5de9487e1b93b1d
Gecko: 0b3c520002ad
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 39.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Repro frequency: 5/5
See attached: screenshot
Reporter | ||
Comment 1•10 years ago
|
||
This issue also reproduces on Flame 2.2.
Result: The closing parenthesis is not placed at the end of "oficial".
Device: Flame 2.2 (KK, 319mb, full flash)
BuildID: 20150303002527
Gaia: 3d188c414e30acc392253d5389a42352fcfbc183
Gecko: c89aad487aa5
Gonk: e7c90613521145db090dd24147afd5ceb5703190
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?][rtl-impact]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?][rtl-impact] → [QAnalyst-Triage+][rtl-impact]
Flags: needinfo?(ktucker)
Comment 2•10 years ago
|
||
If I understand correctly, this is something localizers can fix. ni on them!
Flags: needinfo?(rami223)
Flags: needinfo?(nefzaoui)
Updated•10 years ago
|
Component: Gaia::System → ar / Arabic
Product: Firefox OS → Mozilla Localizations
Comment 3•10 years ago
|
||
Delphine, can you give me the string in English because I tried to find it but with no luck
Flags: needinfo?(rami223)
Updated•10 years ago
|
No longer blocks: system-rtl
Comment 4•10 years ago
|
||
Hey Rami, I think this is actually probably a string in Marketplace! Asking Peiying for confirmation. And if it is, I'm sure she can point you to it :) thanks!
Flags: needinfo?(pmo)
Comment 5•10 years ago
|
||
From what I understand, the RTL development has not started yet. We just finished design review not long ago. let me check
Flags: needinfo?(pmo)
Comment 6•10 years ago
|
||
talked to Wil, the Dev manager on this, and he confirmed what I said. They will scope the work and provide an estimate on when the development can be completed. Please also keep in mind that for RTL, only the Fireplace (Front End) will be RTLed. The rest of the marketplace features will take a lot longer to complete, and won't be able to finish to meet the current roadmap plan.
this is the design: https://wiki.mozilla.org/Marketplace/RTL#Design
Comment 8•10 years ago
|
||
Adding the LocRun2.2 tag in Whiteboard for l10n tracking purposes in a bugzilla dashboard
Whiteboard: [3.0-Daily-Testing][systemsfe] → [3.0-Daily-Testing][systemsfe] LocRun2.2
Comment 9•9 years ago
|
||
Some String still in English and some other in bad Format
Comment 10•9 years ago
|
||
Manel: it seems you are describing multiple bugs in comment 9, correct? If not, how are they linked to this one? You might have to open multiple bugs for this, I think - but first I need more info to understand.
Also, what version are you on? thanks!
Flags: needinfo?(manel.rhaiem92)
Comment 11•9 years ago
|
||
@delphine: I am in the 2.2 Version I made "make install-gaia" First to reach for the Bug in attachment But I found another Bugs where I sent in the attachment.
Flags: needinfo?(manel.rhaiem92)
Comment 12•9 years ago
|
||
I see. Please always file separate bugs for each issue you find ;) (so one issue = one bug)
thanks!
Comment 13•9 years ago
|
||
Is this particular issue resolved? From what I can see on the desktop, it appears it is fixed. I can't find the exact app but I did find apps with parenthesis. It looks fine for this type of issue. Feel free to reopen if issue still exists.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 14•9 years ago
|
||
Lancy: can you please help confirm this is fixed on Firefox OS (comment 13 might not reflect actual results)? thanks!
Flags: needinfo?(yulan.zhu)
Comment 15•9 years ago
|
||
Hi Delphine,
This issue is verified as pass on latest Flame v2.2&Master user build by the STR in comment 0, Parentheses are placed correctly in Arabic language.
See attachment:Verify1_AppNameWithParenthesis_v2.2&Master_Pass.png
Reproducing rate:0/10
Device: Flame 2.2 user build (Pass)
Build ID 20150707162503
Gaia Revision ea11f422b687a982f0a961c9aea7858066561707
Gaia Date 2015-07-02 23:37:50
Gecko Revision https://hg.mozilla.org/releases/mozilla-b2g37_v2_2/rev/52d19a805d35
Gecko Version 37.0
Device Name flame
Firmware(Release) 4.4.2
Firmware(Incremental) eng.cltbld.20150707.195609
Firmware Date Tue Jul 7 19:56:20 EDT 2015
Bootloader L1TC000118D0
Device:Flame master user build (Pass)
Build ID 20150707160207
Gaia Revision 86af137c7f4c41c1185e609339002ab47fd6c640
Gaia Date 2015-07-07 17:02:07
Gecko Revision https://hg.mozilla.org/mozilla-central/rev/9340658848d1
Gecko Version 42.0a1
Device Name flame
Firmware(Release) 4.4.2
Firmware(Incremental) eng.cltbld.20150707.193055
Firmware Date Tue Jul 7 19:31:07 EDT 2015
Bootloader L1TC000118D0
Flags: needinfo?(yulan.zhu)
Updated•9 years ago
|
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+][rtl-impact] → [QAnalyst-Triage+][rtl-impact][MGSEI-Triage+]
Comment 16•9 years ago
|
||
Awesome, thanks Lancy! :)
You need to log in
before you can comment on or make changes to this bug.
Description
•