Closed Bug 1351813 Opened 3 years ago Closed 3 years ago

Font size is small

Categories

(Webtools :: Pontoon, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: dr.khaled.hosny, Unassigned, Mentored)

References

Details

The font size of the translation fields (the text editing area, the string list), is rather small, for Arabic at least. I had to zoom to about 120% to get comfortable font size, but this makes everything bigger which is undesirable.
While I can't read Arabic, I can totally understand your point by simply comparing Arabic translations in the textarea and string list with e.g. German:
https://pontoon.mozilla.org/ar/pontoon-intro/all-resources/
https://pontoon.mozilla.org/de/pontoon-intro/all-resources/

It seems like the font-family we use (Open Sans) uses relatively small Arabic characters compared to Latin. 

Implementation note:
We can increase the font size for Arabic only, by hooking to the [lang] attribute like this:
https://github.com/mozilla/pontoon/commit/b48239dfa4853b1653d3f63bcf40d45b0430a876

Also, I might have found another bug while looking at this. If you go to the Locales tab, shouldn't the Arabic translations be aligned right?
https://pontoon.mozilla.org/de/pontoon-intro/all-resources/?string=33052
Mentor: m
Priority: -- → P2
It should(In reply to Matjaz Horvat [:mathjazz] from comment #1)
> While I can't read Arabic, I can totally understand your point by simply
> comparing Arabic translations in the textarea and string list with e.g.
> German:
> https://pontoon.mozilla.org/ar/pontoon-intro/all-resources/
> https://pontoon.mozilla.org/de/pontoon-intro/all-resources/

Open Sans has no Arabic support at all, so it will be whatever fallback font that the browser will end up selecting and most Arabic fonts are, unfortunately, designed smaller compared to Latin.

> Also, I might have found another bug while looking at this. If you go to the
> Locales tab, shouldn't the Arabic translations be aligned right?
> https://pontoon.mozilla.org/de/pontoon-intro/all-resources/?string=33052

Yes it should, it should also be right aligned in the the strings list on the left:
https://pontoon.mozilla.org/ar/pontoon-intro/all-resources/?string=33050
IIRC, Pootle makes Arabic text 120% the normal size.
(In reply to Khaled Hosny from comment #2)
> Open Sans has no Arabic support at all, so it will be whatever fallback font
> that the browser will end up selecting and most Arabic fonts are,
> unfortunately, designed smaller compared to Latin.

Good point. :-)
Khaled, what are your thoughts on the proposed improvement deployed to stage?
https://mozilla-pontoon-staging.herokuapp.com/ar/engagement/snippets/2017/may2017_b.lang/

For reference, this is the current state:
https://pontoon.mozilla.org/ar/engagement/snippets/2017/may2017_b.lang/
Flags: needinfo?(khaledhosny)
Looks good, much better. Thanks.
Flags: needinfo?(khaledhosny)
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Commit pushed to master at https://github.com/mozilla/pontoon

https://github.com/mozilla/pontoon/commit/7bf85fba78e0f04d076465e27e9b9c429def6ffb
Bug 1351813: Avoid using !important in CSS

Explicitly set Arabic and Persian-specific font-size.
You need to log in before you can comment on or make changes to this bug.