#20454 closed Bug (fixed)
Urdu is RTL
| Reported by: | Axel Hecht | Owned by: | pike |
|---|---|---|---|
| Component: | Core (Other) | Version: | dev |
| Severity: | Normal | Keywords: | |
| Cc: | pike, Jannis Leidel | Triage Stage: | Accepted |
| Has patch: | no | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
In #15300, Urdu was added, but not added to the list of RTL locales. But it is.
References: https://github.com/django/django/blob/master/django/contrib/admin/locale/ur/LC_MESSAGES/django.po shows if you select bidi text, and http://en.wikipedia.org/wiki/Urdu says so.
I'll create a PR
Change History (7)
comment:1 by , 12 years ago
| Triage Stage: | Unreviewed → Accepted |
|---|
comment:2 by , 12 years ago
I had this alllllll written out, but apparently I didn't submit my comment.
I checked, there are some candidates of languages that are also written in RTL, but also in other scripts, and the ones in the tree use non-RTL scripts.
The PR for this ticket is https://github.com/django/django/pull/1142.
I didn't see any existing tests for RTL languages.
comment:4 by , 12 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:5 by , 12 years ago
Urdu is still flagged as not bidi in LANG_INFO here: https://github.com/django/django/blob/master/django/conf/locale/__init__.py
Looks like a good idea. Marking as Accepted.
Are there tests for RTL languages?