nextcloud-libraries / nextcloud-l10n

Nextcloud L10n helpers for apps and libraries https://npmjs.org/@nextcloud/l10n
https://nextcloud-libraries.github.io/nextcloud-l10n/
GNU General Public License v3.0
6 stars 4 forks source link

Update list of RTL languages #803

Open nickvergessen opened 1 week ago

nickvergessen commented 1 week ago

isRTL is not fully reporting all languages as RTL that the server has listed as RTL:

As per discussion in #801 and chat, the following languages should be listed as RTL:

$rtlLanguages = [
    'ar', // Arabic
    'fa', // Persian
    'he', // Hebrew
    'ps', // Pashto,
    'ug', // 'Uyghurche / Uyghur
    'ur_PK', // Urdu
];

Others can be listed, but at least those should be, but ur_PK which will be available as ur-PK in the head lang attribute is missing.

nickvergessen commented 1 week ago

Also the function currently seems to consider locale for uz-AF but we agreed that RTL handling should be language based.