search

pnarimani / RTLTMPro

Right-To-Left Text Mesh Pro for Unity. This plugin adds support for Persian and Arabic languages to TextMeshPro.
MIT License
526 stars 100 forks source link

Regression In v3.4.3, Characters Go Missing, Worked Fine In v3.4.2 (only shows up in some fonts) #108

Open MotasemZakarneh opened 1 year ago

MotasemZakarneh commented 1 year ago

Tested with two versions its broken in the latest version, and it is a weird bug does not show up in all fonts.

First here is the font

Mirza-Regular SDF.zip

The sentence i was testing with is: السلام عليكم when i used RTLTMPro.v3.3.0.unitypackage it worked just fine

image

but using v3.4.3.unitypackage it is broken (latest at the moment), it looks like this:

image

Literally, the only thing i did was install package, and remove package. Note: not sure if it effects anything, but in both tests i moved the package in a sub folder

This is the range i used for the font asset creation and its 2kx2k atlas resolution with padding of 5

48-57,1548,1563,1567,1570-1594,1600-1641,1648,1662,1670,1688,1705,1711,1740,1776-1785,64343-64345,64379-64381,64395-64401,64403-64405,64509-64511,64606-64611,65154-65164,65166-65170,65172-65176,65178-65180,65182-65184,65186-65188,65190-65192,65194-65204,65206-65208,65210-65212,65214-65216,65218-65220,65222-65224,65226-65228,65230-65232,65234-65236,65238-65240,65242-65244,65246-65248,65250-65252,65254-65256,65258-65260,65262-65264,65266-65276,32-126,160-255,8192-8207,8210-8226,8230,8234-8240,8242-8244,8249-8250,8252,8254,8260,8286,8298-8303,8364,8482,9633,0600-06FF,0750-077F,08A0-08FF,FB50-FDFF,FE70-FEFF

Also, every time i reimported, i recreated the font assets.

Note: using Segiou (the one that came in the package) it works and using Amiri font (found on google) also worked. but i can not imagine it is an issue with the font, because in the older version mirza simply worked, i guess there are some requirements on which font to use ?

Note 2: i did one more test using RTLTMPro.v3.4.2.unitypackage and it worked, without the issue, so the change appeared only in the latest version.

Note 3: upon further testing it happens in other fonts, it happened in Arslan font as well

(A) Arslan Wessam A (A) Arslan Wessam A.zip

marcozakaria commented 10 months ago

Same issue with me was you able to solve it @MotasemZakarneh @hk1ll3r