Changes between Initial Version and Version 4 of Ticket #10436


Ignore:
Timestamp:
Mar 31, 2009, 6:19:43 PM (15 years ago)
Author:
Ramiro Morales
Comment:

(made description more clear)

Legend:

Unmodified
Added
Removed
Modified

  • Ticket #10436

    • Property Summary Application names i10n in the admin app broken → Application names i18n in the admin app broken
    • Property Triage Stage Unreviewed → Accepted
    • Property Cc Ben Spaulding added

  • Ticket #10436 – Description

    initial v4  
    55}}}
    66
    7 as we currently do in some admin templates shows a decision has been made to provide developers the means to translate application names. The problem is this feature was DOA (r3270, `django/contrib/admin/templates/admin/index.html`, similar template code was copied since then to another file: r8474, `app_index.html`)
     7as we currently do in some admin templates shows a decision has been made to provide developers the means to translate application names.
     8
     9Problem is this feature was DOA (r3270, `django/contrib/admin/templates/admin/index.html`, similar template code was copied since then to another file: r8474, `app_index.html`)
    810
    911See relevant [http://groups.google.com/group/django-users/browse_frm/thread/f8fcbe25e3327c5d django-users] and [http://groups.google.com/group/Django-I18N/browse_frm/thread/bdcdaf433bfcd144 django-i18n] threads for the details.
    1012
    11 The attached patch implements the manual steps outlined by the user ''patrikk'' in the first thread with the following enhancements:
     13The attached patch implements as a fix the awkward manual steps needed to workaround this bug outlined by the user ''patrikk'' in the first thread with the following enhancements:
    1214
    1315 * `index`, `app_index`: Moved the markup-as-translatable of app names from the template to the view code so the translators don't need to translate a second title()'d version of every app name.
    14  * Consistent use of the capfirst filter over the app names in the breadcrumbs of the different views
    15  * Don't touch `base_site.html` in order to provide for translatability of the app name in the <title> HTML tag for the application-specific model list view, implement the needed change in the relevant view (`app_index`) instead (replacing a "%s administration" msgid with a more translator-friendly "%(app_label)s administration" one in the process). This spare translators the need to translate both a "flatpages" and a "flatpages administration" literal.
    16  * Extended fixes to other templates that also need them (`delete_confirm.html`,  `object_history.html`)
     16 * Consistent use of the capfirst filter over the app names in the breadcrumbs of the different views.
     17 * Don't touch `base_site.html` in order to provide for translatability of the app name in the <title> HTML tag for the application-specific model list view. Implement the needed change in the relevant view (`app_index`) instead (replacing a "%s administration" msgid with a more translator-friendly "%(app_label)s administration" one in the process). This will spare translators having to translate both "<appaname>" and a "<appaname administration" literals.
     18 * Extend above fixes to other templates that also need them (`delete_confirm.html`,  `object_history.html`)
    1719
    1820'''Note 1:''' The `change_form.html` template ran the app label (in addition to the `capfirst` filter) through the `escape` filter. This has been dropped because it seems inconsistent with what is done in rest of the templates, and because the app name isn't user-provided content.
    1921
    20 '''Note 2:''' There seems to be some inconsistency on how the app name is .title()'d for the name that appear in the caption of model tables (`index` and `app_index` views`) and `|capfirst`'d for use in HTML title and breadcrumbs. This has been left untouched to avoid introducing a backward incompatible change.
     22'''Note 2:''' There seems to be some inconsistency on how the app name is a) .title()'d for the name that appear in the caption of model tables (`index` and `app_index` views`) and b) |capfirst'd for use in HTML title and breadcrumbs. This has been left untouched to avoid introducing a backward incompatible change.
    2123
    22 Note that several of the problems exposed above had already been reported in #9273 and that some of the presentation-related (like calling .title() and/or capfirst() in view code could be moved to templates if/when #5972 gets committed (will try to work on adding test+docs to it.)
     24'''Note 3:''' A further enhancement would be to move presentation-related munging (like calling .title() and/or capfirst() in view code) from views to templates if/when #5972 gets committed (will try to work on adding test+docs to it) but meanwhile this fix can solve this bug now with what we have at hand.
     25
     26Several of the problems exposed above had already been reported in #9273.
Back to Top