MoathOthman / MOLH

Localization helper for iOS apps mainly focusing on the LTR/RTL issue
MIT License
117 stars 36 forks source link

The "fr" language is not a RTL language #44

Closed medHajajate closed 4 years ago

medHajajate commented 4 years ago

Hi, In your class MOLHLanguage, the RTL language array contains the "fr" language, you should remove it, as you know the fr isn't a RTL language,

Best Regards.

MoathOthman commented 4 years ago

Correct, that was supposed to be fa. Will fix it asap

Sent from my iPhone

On Nov 21, 2019, at 8:18 PM, Med Hajajate notifications@github.com wrote:



Hi, In your class MOLHLanguage, the RTL language array contains the "fr" language, you should remove it, as you know the fr isn't a RTL language,

Best Regards.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHubhttps://github.com/MoathOthman/MOLH/issues/44?email_source=notifications&email_token=AAI43B37UIPEAMKYJBACYDTQU27H5A5CNFSM4JQFTIA2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4H3FKD2A, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAI43BYVQEMUGEHAWX6UT2DQU27H5ANCNFSM4JQFTIAQ.

MoathOthman commented 4 years ago

fixed now , thnx for reporting.