quran / quran_android

a quran reading application for android
http://android.quran.com
GNU General Public License v3.0
2.02k stars 890 forks source link

arabic font issues on pre-MR2 devices #580

Closed ahmedre closed 9 years ago

ahmedre commented 9 years ago

arabic font is not properly rendered pre-MR2. several reports both on MR1 and on 4.x (i could repro on 4.0.3). on 4.0.3, even the default font runs into each other, but not as badly as the new font. unfortunately, there's likely little we can do about this besides just disabling the new font on those devices.

opening this here just in case anyone has any ideas.

ahmedre commented 9 years ago

how it looks like in 4.0.3 (without the uthmani font, i.e. how it used to always be): screenshot_2015-10-13-23-44-02

how it looks like 4.0.3 with the uthmani font: screenshot_2015-10-13-23-47-27

and with naskh: screenshot_2015-10-13-23-10-21

some interesting points:

  1. it's been broken like this for ages and no one has complained (i.e. like the first screenshot).
  2. people are now also complaining about the overlap of words on top of each other (ex, this screenshot from an email from a brother):

screenshot_2015-10-09-11-07-31

what i've currently done in dev branch is to disable this feature pre-MR2 (i.e. use the old font). while technically, this returns things back the way they were for pre-MR2 people using the app, it doesn't address the fact that things have been broken all along (as the first screenshot would seem to indicate).

ahmedre commented 9 years ago

ok, will keep this dev fix. please feel free to re-open if you have any good ideas on how to work around this issue!