Stephen Finucane <[email protected]> writes: > Two issues here. Firstly, the use of the 'USE_I18N'. The Django docs > describe this as such: > > A boolean that specifies whether Django’s translation system should > be enabled. This provides an easy way to turn it off, for performance. > If this is set to False, Django will make some optimizations so as not > to load the translation machinery. > > We don't do translations and won't until such a time as someone comes > asking for them. Optimize things accordingly by setting 'USE_I18N' to > False and removing the now-unnecessary 'LANGUAGE_CODE' setting. > > Secondly, the use of en-AU is a bit of a lie since our UI is actually > written in US English (or should be). The primary reason for a lang tag > to be present is to assist screenreaders and other accessibility tools, > so make their lives easier by reflecting the truth.
I am not sold on the primacy of US English, and from a historical point of view the UI was absolutely written in Australian English, but before I go down that rabbit-hole, what are the actual implications of setting en-AU vs en-US for anything that might parse the web page? If there's actually a case where the difference would matter, I'd be much happer to consider changing it. I don't suppose there's a plain "en" language code? Daniel > > Signed-off-by: Stephen Finucane <[email protected]> > Cc: Andrew Donnellan <[email protected]> > Cc: Daniel Axtens <[email protected]> > --- > patchwork/settings/base.py | 4 +--- > templates/base.html | 2 +- > 2 files changed, 2 insertions(+), 4 deletions(-) > > diff --git patchwork/settings/base.py patchwork/settings/base.py > index b86cdc27..86612d7a 100644 > --- patchwork/settings/base.py > +++ patchwork/settings/base.py > @@ -36,9 +36,7 @@ MIDDLEWARE = [ > > TIME_ZONE = 'Australia/Canberra' > > -LANGUAGE_CODE = 'en-au' > - > -USE_I18N = True > +USE_I18N = False > > TEST_RUNNER = 'django.test.runner.DiscoverRunner' > > diff --git templates/base.html templates/base.html > index 40b6cda6..27db7f52 100644 > --- templates/base.html > +++ templates/base.html > @@ -1,6 +1,6 @@ > {% load static %} > <!DOCTYPE html> > -<html xmlns="http://www.w3.org/1999/xhtml" lang="en-AU"> > +<html xmlns="http://www.w3.org/1999/xhtml" lang="en-US"> > <head> > <meta http-equiv="Content-Type" content="text/html; charset=UTF-8"/> > <title>{% block title %}Patchwork{% endblock %} - Patchwork</title> > -- > 2.21.0 _______________________________________________ Patchwork mailing list [email protected] https://lists.ozlabs.org/listinfo/patchwork
