1140977 - [Messages] Bottom few pixels of Arabic characters cut off

Status: RESOLVED DUPLICATE of bug 1150449

(Firefox OS Graveyard :: Gaia::SMS, defect)

Description

[Steve Chung [:steveck]]

In https://github.com/mozilla-b2g/gaia/pull/28444#issuecomment-77583214, we found some Arabic characters will be cut because of inappropriate ratio of font-size/line height. Maybe it's time to revisit this issue in message app.

Comment 1

[Steve Chung [:steveck]]

WIP: https://github.com/steveck-chung/gaia/commit/e2e19b1aff0ec80645a777bcd1ff63d7a0a8ec7e

Since bug 1138350 will fix thread list RTL ellipsis display issue first, this patch will apply thread list fixing based on bug 1138350.

Comment 2

[Steve Chung [:steveck]]

Attachment: text layout

Hi Fang, in order to display the Arabic text correctly, I adjust the contact name and number line-height to avoide certain char been cut(mostly because of this char "ي"). As you can see it will take much more space than orignal layout and the operator title might be very close to bottom line. Can you accept this text layout? Please note that I didn't left any space for padding/margin so there's not much room that we can reduce the space properly.

Comment 3

[Fang Shih [:fang]]

Comment on attachment 8586707 [details]
text layout

Hi Steve, 
Thanks for the adjustment. I think the spacing looks fine between contact name and number in Arabic text. But I'm just a bit worry the bottom line may look way to close the text, especially in English text, while the spacing looks huge between name and number, but not much space left for the bottom part. Is there anything we can do about it by adding more space for bottom line? Thanks

Comment 4

[Steve Chung [:steveck]]

Attachment: text_layout_ver2

I tried negtive margin to reduce the space between the text, it seems still safe from the text truncate, but now quite sure if it's fine for all the cases.

Comment 5

[Fang Shih [:fang]]

Comment on attachment 8587148 [details]
text_layout_ver2

It looks good! Thank a lot!

Comment 6

[Steve Chung [:steveck]]

Per bug 1133137 comment 15, some fonts is missing in pvt and I can not reproduce this issue after using v18D base image. Close this one since bug 1152420 will keep tracking the missing fonts in pvt build.