Closed Bug 1139209 Opened 9 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)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(b2g-v2.2 verified, b2g-master verified)

VERIFIED FIXED
Tracking Status
b2g-v2.2 --- verified
b2g-master --- verified

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
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)
QA Whiteboard: [QAnalyst-Triage?][rtl-impact] → [QAnalyst-Triage+][rtl-impact]
Flags: needinfo?(ktucker)
If I understand correctly, this is something localizers can fix. ni on them!
Flags: needinfo?(rami223)
Flags: needinfo?(nefzaoui)
Component: Gaia::System → ar / Arabic
Product: Firefox OS → Mozilla Localizations
Delphine, can you give me the string in English because I tried to find it but with no luck
Flags: needinfo?(rami223)
No longer blocks: system-rtl
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)
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)
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
Clearing the ni on Ahmed since Rami answered ;)
Flags: needinfo?(nefzaoui)
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
Attached file Bugs1139209.zip
Some String still in English and some other in bad Format
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)
@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)
I see. Please always file separate bugs for each issue you find ;) (so one issue = one bug)
thanks!
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
Lancy: can you please help confirm this is fixed on Firefox OS (comment 13 might not reflect actual results)? thanks!
Flags: needinfo?(yulan.zhu)
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)
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+][rtl-impact] → [QAnalyst-Triage+][rtl-impact][MGSEI-Triage+]
Awesome, thanks Lancy! :)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: