Closed
Bug 1035471
Opened 10 years ago
Closed 7 years ago
[B2G][2.0][l10n][SMS] Tamil: "(No Recipient)" string is cut off in the messages list.
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect, P3)
Tracking
(tracking-b2g:backlog, b2g-v2.0 affected)
People
(Reporter: Marty, Unassigned)
References
Details
(Whiteboard: LocRun2.0)
Attachments
(4 files)
Description: "(No Recipient)" string is cut off along the bottom of the text line in Tamil Repro Steps: 1) Update a Flame device to BuildID: 20140707000200 2) Change language to Tamil (தமிழ்) 3) Restart device 4) Open a new SMS message and save a draft without a recipient. 5) Note the "(No recipient)" string in the messages list. Actual: "(No Recipient)" translated string is cut off. Expected: No cut-off occurs. Environmental Variables: Device: Flame 2.0 BuildID: 20140707000200 Gaia: ef67af27dff3130d41a9139d6ae7ed640c34d922 Gecko: f53099796238 Version: 32.0a2 Firmware Version: v122 2.0 - Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0 Keywords: Tamil, SMS, Truncation Repro frequency: 100% Link to failed test case: https://moztrap.mozilla.org/manage/case/12588/ See attached: Screenshot ------------------------------------------------------- This issue does not occur on 1.4 Flame. Environmental Variables: Device: Flame 1.4 BuildID: 20140630000201 Gaia: aa896d5db1b4929f3bf31a0f4bb7de50530222a8 Gecko: 8cba60bc12ef Version: 30.0 (1.4) Firmware Version: v122 User Agent: Mozilla/5.0 (Mobile; rv:30.0) Gecko/30.0 Firefox/30.0 Tamil was not an available language.
Reporter | ||
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage+][lead-review+]
Comment 1•10 years ago
|
||
I have checked with Keon device with Build(20140722223137), It is not to do with Translation, because the localizied Tamil strings(date & No receipent) are apearing fine except cut off. The strings bootom portion(tiny) is cut off. It may be due to default Tamil font size/height. Please advice us.
Flags: needinfo?(mshuman)
Reporter | ||
Comment 2•10 years ago
|
||
Moving this to Gaia:SMS. Tamil appears to be the only language affected here. This issue does NOT occur on Flame 2.1. "(No Recipient)" translated string appears appropriately. Device: Flame Master Build ID: 20140725040205 Gaia: 3a06aa58245eaf848242d6d1497c1af536fffabd Gecko: 6c0971104909 Version: 34.0a1 (Master) Firmware Version: v122 User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0 ------------------------------------------------------------------- This issue DOES occur on Open-C 2.0 and Open-C 2.1. "(No Recipient)" translated text is cut off at the bottom. Environmental Variables: Device: Open_C Master Build ID: 20140725040205 Gaia: 3a06aa58245eaf848242d6d1497c1af536fffabd Gecko: 6c0971104909 Version: 34.0a1 (Master) Firmware Version: P821A10V1.0.0B06_LOG_DL User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0 Environmental Variables: Device: Open_C 2.0 Build ID: 20140725000201 Gaia: 9b6d7357031f2412b18a2fb140d5c974842d4393 Gecko: fbb3b8be8f6c Version: 32.0 (2.0) Firmware Version: P821A10V1.0.0B06_LOG_DL User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0 ------------------------------------------------------------------- I was unable to test this on the Buri device, as Tamil is not an included language in the Buri builds.
Component: ta / Tamil → Gaia::SMS
Flags: needinfo?(mshuman)
Product: Mozilla Localizations → Firefox OS
Reporter | ||
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage+][lead-review+] → [QAnalyst-Triage?][lead-review+]
Flags: needinfo?(ktucker)
Comment 3•10 years ago
|
||
Do we have different fonts on Flame and Open C? Gaia::SMS is not the right component for this but I don't know who we can NI for this...
Comment 4•10 years ago
|
||
For your information, for tamil we recently changed the fonts to noto. It is changed for keon, and yet to change for flame. Flame is still using Lohit fonts.
Comment 5•10 years ago
|
||
Ok, so this is probably the reason of the discrepancy. Any ETA for Flame?
Comment 6•10 years ago
|
||
I am not very sure, but look at this, https://bugzilla.mozilla.org/show_bug.cgi?id=1022311#c2
Comment 7•10 years ago
|
||
Hey Pike, do you know if this bug is INVALID because we don't use the correct fonts on Flame? Do you know a path forward?
Flags: needinfo?(l10n)
Comment 8•10 years ago
|
||
I updated my flame to 2.0, and uploaded the Noto fonts, and removed the Lohit fonts, and rebooted, and I still see the same cut-off behavior.
Flags: needinfo?(l10n)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?][lead-review+] → [QAnalyst-Triage+][lead-review+]
Flags: needinfo?(ktucker)
Comment 9•10 years ago
|
||
Pike, do you see the exact same thing than attachment 8451969 [details]?
I'd like to be sure we have the same outcome with the right fonts on all devices, because the differences are puzzling me so far.
Pike, do you know if this would be a release blocker?
Flags: needinfo?(l10n)
Comment 10•10 years ago
|
||
Here's what I see locally, edited-out my personal text messages on there.
Flags: needinfo?(l10n)
Comment 11•10 years ago
|
||
Thanks, so it's the exact same font, so I can safely say the problem arises with the new font. NI me to investigate further and find possible solutions.
Flags: needinfo?(felash)
Comment 12•10 years ago
|
||
Actually, we see that the "today" text is also cut off in the bottom.
Comment 13•10 years ago
|
||
I tried with both fonts and I see the same issue in both fonts. So maybe I'm not looking at the right place. Marty, can you please put a screenshot of a case where this works appropriately?
Flags: needinfo?(felash) → needinfo?(mshuman)
Keywords: qawanted
Comment 14•10 years ago
|
||
I see the same on a Open C 1.3 upgraded to a v2.0 Gecko+Gaia (coming from a Flame build).
Reporter | ||
Comment 15•10 years ago
|
||
I do not see the cut-off issue in Flame 2.1, and have attached a screenshot. Environmental Variables: Device: Flame Master Build ID: 20140729040211 Gaia: fadfafa17f5175203b8b9457bfb95e5816f54f58 Gecko: b17cad2d1e5e Version: 34.0a1 (Master) Firmware Version: v123 User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0
Flags: needinfo?(mshuman)
Comment 16•10 years ago
|
||
I'm not sure exactly which elements we're looking at here, but I suspect the issue is that the associated CSS is setting an explicit line-height that is only slightly larger than the element's font-size. I see plenty of such styling in https://github.com/mozilla-b2g/gaia/blob/v2.0/apps/sms/style/sms.css, for example. This may lead to clipping (at least if overflow-y is set to 'hidden') for non-Latin fonts where ascenders and/or descenders may extend considerably further than is typical for Latin script. Some of the CSS there (e.g. at [1]) even sets line-height equal to font-size, which is likely to lead to unwanted clipping even for Latin script when accented capital letters are present. With styles based on that fragment, I see clipping in a testcase like data:text/html;charset=utf-8,<div style="font-family:Fira Sans OT;padding: 0 0 0.7rem; font-size:1.5rem;line-height:1.5rem;overflow:hidden">Fira S&%23x301;ans&%23x301; So I suspect this is primarily an app styling issue: the designers have used overly rigid styling that is based on the dimensions of the English text, and other scripts simply don't fit in the same space. [1] https://github.com/mozilla-b2g/gaia/blob/v2.0/apps/sms/style/sms.css#L776
Comment 17•10 years ago
|
||
Hi Team, Please confirm us(l10n-ta), is this also reason for more truncations in fxOS V2.x than FxOS V1.3 ? Since we are reducing the words in our translation(it looks funny) to reduce the truncation issues for fxOS V2.0 .
Flags: needinfo?(mshuman)
Flags: needinfo?(jfkthame)
Comment 18•10 years ago
|
||
Martin> Thanks Jonathan> Yep, I agree this is the likely cause. But I try to see where is the issue to fix it. It's also strange that it does not happen in other versions with the same font, since we did not change anything in this regard, so I want to understand this before doing anything else. Kumar> I don't think this is the reason for more truncations. I still want to know if this is a likely 2.0 blocker, or if a 2.1 timeframe is good enough.
Comment 19•10 years ago
|
||
Release Management: can you check with product to know if this makes a 2.0 blocker?
blocking-b2g: --- → 2.0?
Comment 20•10 years ago
|
||
(In reply to அருண் குமார்-Arun Kumar from comment #17) > Hi Team, > > Please confirm us(l10n-ta), is this also reason for more truncations in fxOS > V2.x than FxOS V1.3 ? > > Since we are reducing the words in our translation(it looks funny) to reduce > the truncation issues for fxOS V2.0 . Note that this is about clipping the bottom (or top) of the glyphs, which is a separate issue from truncation at the end of the text because of being too long horizontally. You may be able to fix that kind of truncation (text too long for the space provided) by modifying the translation to use shorter words and phrases. (In reply to Julien Wajsberg [:julienw] from comment #18) > Jonathan> Yep, I agree this is the likely cause. But I try to see where is > the issue to fix it. It's also strange that it does not happen in other > versions with the same font, since we did not change anything in this > regard, so I want to understand this before doing anything else. Comparing the screenshots from attachment 8451969 [details] (with clipping) and attachment 8464055 [details] (no clipping), it looks to me like there's a slight difference in line spacing: the latter screenshot has about 1px more space between the slanted "(no recipient)" text and the time/message text below it. That 1px probably corresponds to the difference between clipped and not-clipped. But I don't know exactly what styles are controlling this. Oh, here's another thought: does the change here coincide with an update to the Fira Sans fonts? If the vertical metrics of *those* fonts have changed, that could explain slightly different vertical positioning of the text. Note that in the later screenshot (where the Tamil is not clipped at the bottom), the baseline of all text -- even the current time in the status bar at the very top -- seems to be shifted slightly higher. That could be affected by the metrics of the default font, I think.
Flags: needinfo?(jfkthame)
Reporter | ||
Comment 21•10 years ago
|
||
Looks like julienw and jfkthame have answered this in comment #18 and comment #20.
Flags: needinfo?(mshuman)
Comment 22•10 years ago
|
||
Triage Question - What user impact is there in this bug to understandability of the Tamil language here?
Flags: needinfo?(thangam.arunx)
Comment 24•10 years ago
|
||
It seems the screenshot from comment 15 satisfied the QA-Wanted tag in comment 13, removing tag
Keywords: qawanted
Comment 25•10 years ago
|
||
This would be a blocker for Tamil if, because of the clipping, the text becomes completely unintelligible. I doubt that, but we need the Tamil team to confirm that it is still understandable. Considering it's a shipping locale, if the sense is lost, it would then be a blocker.
Flags: needinfo?(lebedel.delphine)
Comment 26•10 years ago
|
||
We can proceed, since attachment 8464055 [details] looks fine.
Arun Prakash, kindly put your words here.
Flags: needinfo?(thangam.arunx) → needinfo?(arunprakash.pts)
Comment 28•10 years ago
|
||
This is not a blocker. User can read the above text.
Flags: needinfo?(arunprakash.pts)
Assignee | ||
Updated•9 years ago
|
blocking-b2g: backlog → ---
tracking-b2g:
--- → backlog
Updated•9 years ago
|
Priority: -- → P3
Comment 29•9 years ago
|
||
Given our issues with arabic fonts (UI vs non-UI flavors), could we have the same happening Tamil? Is there a flavor of a Tamil font that has the same size characteristics than usual latin fonts?
Comment 30•9 years ago
|
||
(In reply to Julien Wajsberg [:julienw] from comment #29) > Given our issues with arabic fonts (UI vs non-UI flavors), could we have the > same happening Tamil? Is there a flavor of a Tamil font that has the same > size characteristics than usual latin fonts? We currently have Noto Sans Tamil in moztt. There is a Noto Sans Tamil UI variant that is designed to be vertically more compact, so it would probably help here.
Comment 31•7 years ago
|
||
Mass closing of Gaia::SMS bugs. End of an era :(
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
Comment 32•7 years ago
|
||
Mass closing of Gaia::SMS bugs. End of an era :(
You need to log in
before you can comment on or make changes to this bug.
Description
•