#19600 closed task (blessed) (fixed)
Core should know which locales are RTL
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Priority: | normal | Milestone: | 3.4 |
| Component: | I18N | Version: | |
| Severity: | normal | Keywords: | has-patch |
| Cc: |
Description
Currently an RTL locale needs to specify $text_direction = 'rtl'; in a $locale.php file. In the interest of eliminating the need for $locale.php files, core should keep a list of RTL locales.
$text_direction would remain functional, and we're not killing $locale.php -- just trying to cover its primary uses.
While a simple array of RTL locales can be maintained now, there might reach a point where it will be easier to include locales/locales.php in core. (Indeed, mu_dropdown_languages() would benefit from this.) In that case, RTL could be read from there.
Attachments (1)
Change History (8)
SergeyBiryukov — 17 months ago
comment:1
SergeyBiryukov — 17 months ago
- Keywords has-patch added
comment:2
SergeyBiryukov — 17 months ago
- Owner set to nacin
- Resolution set to fixed
- Status changed from new to closed
In [19755]:
I have a strong feeling that [19755] will evolve more as 3.4 continues. We'll work with this for now.
comment:25
nacin — 16 months ago
In [19836]:

The patch adds the locales which currently set text_direction in $locale.php.
There are some more (seemingly unmaintained) locales in GlotPress marked as RTL: dv, fa_AF, ha, ps, uz_UZ, yi.
ckb and ug_CN are not marked as RTL in GlotPress, but still set text_direction in $locale.php.