Opened 17 years ago

Closed 14 years ago

#5519 closed (wontfix)

Capitalized names for czech language

Reported by: Petr Marhoun <petr.marhoun@…> Owned by: Marc Garcia
Component: Internationalization Version: newforms-admin
Severity: Keywords: nfa-someday
Cc: Triage Stage: Design decision needed
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

In Django this kind of translations is used for my language:

msgid "Add %s"
msgstr "%s: přidat"

msgid "Change %s"
msgstr "%s: změnit"

So "Add contact" is translated as "kontakt: přidat" - title is not capitalized. So I would like if admin capitalizes all titles. It is possible, I use it in my branch.

I thought that this kind of change should be sent to django-developers mailing list first. But there was no answer so there is a ticket.

I would like to create patch - if you think that it can be applied.

Attachments (2)

06-support-for-cs.diff (4.2 KB ) - added by Petr Marhoun <petr.marhoun@…> 17 years ago.
01-support-for-cs.diff (3.9 KB ) - added by Petr Marhoun <petr.marhoun@…> 17 years ago.
new version - after autoescape merge (depends only on #5425 now)

Download all attachments as: .zip

Change History (8)

comment:1 by James Bennett, 17 years ago

Triage Stage: UnreviewedDesign decision needed

by Petr Marhoun <petr.marhoun@…>, 17 years ago

Attachment: 06-support-for-cs.diff added

comment:2 by Petr Marhoun <petr.marhoun@…>, 17 years ago

Has patch: set
Keywords: admin titles removed
Patch needs improvement: set
Summary: [newforms-admin] - capitalized titlesCapitalized names for czech language

I want to fix these problems:

#: contrib/admin/options.py:524
#, python-format
msgid "Add %s"
msgstr "%s: přidat"

#: contrib/admin/templates/admin/change_list.html:18
#, python-format
msgid "Add %(name)s"
msgstr "%(name)s: přidat"

#: contrib/admin/options.py:599
#, python-format
msgid "Change %s"
msgstr "%s: změnit"

#: contrib/admin/views/main.py:150
#, python-format
msgid "Select %s"
msgstr "%s: vybrat"

#: contrib/admin/views/main.py:150
#, python-format
msgid "Select %s to change"
msgstr "%s: vybrat pro změnu"

#: contrib/admin/templates/admin/pagination.html:10
#, python-format
msgid "%(count)s %(verbose_name)s"
msgstr "%(verbose_name)s: %(count)s"

#: contrib/admin/templates/admin/pagination.html:12
#, python-format
msgid "%(count)s %(verbose_name_plural)s"
msgstr "%(verbose_name_plural)s: %(count)s"

The last two are from #5425 so it depends on resolution of the patch.

I am sorry, my patch depends on some other patches - it is for simple merging. So my patch can't be applied without problem. But I would like to prepare proper patch, if it would be accepted.

by Petr Marhoun <petr.marhoun@…>, 17 years ago

Attachment: 01-support-for-cs.diff added

new version - after autoescape merge (depends only on #5425 now)

comment:3 by Petr Marhoun <petr.marhoun@…>, 17 years ago

Patch needs improvement: unset

comment:4 by Karen Tracey <kmtracey@…>, 17 years ago

Keywords: nfa-someday added

Old admin looks to be no different from newforms-admin for this issue, so this should not block merge.

comment:5 by Marc Garcia, 15 years ago

Owner: changed from nobody to Marc Garcia

comment:6 by Malcolm Tredinnick, 14 years ago

Resolution: wontfix
Status: newclosed

This is really in conflict with how we want to proceed in #5518. If a translation (or other) is providing a string, we aren't going to mess around with the capitalization. The translator will know best what they want to do.

Note: See TracTickets for help on using tickets.
Back to Top