Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Translatable breadcrumb #24

Open
pix666 opened this issue Apr 6, 2015 · 2 comments
Open

Translatable breadcrumb #24

pix666 opened this issue Apr 6, 2015 · 2 comments

Comments

@pix666
Copy link

pix666 commented Apr 6, 2015

Breadcrumbs in 'admin/change_form.html' includes "app_label", which can't be translated:

{% block breadcrumbs %}
<div class="breadcrumbs">
    <a href="{% url 'admin:index' %}">{% trans 'Home' %}</a> &rsaquo; 
    <a href="../">{{ app_label|capfirst|escape }}</a> &rsaquo; 
    {{ opts.verbose_name|capfirst }}
</div>
{% endblock %}

I think you should replace it with:

{% firstof opts.app_config.verbose_name app_label|capfirst|escape %}
@lazybird
Copy link
Owner

lazybird commented Dec 9, 2015

I would suggest to create a pull request for this so that the change can be reviewed collectively.
Thanks for the note.

@niccolomineo
Copy link

niccolomineo commented Oct 21, 2020

I see the verbose_name was added in master, but the most recent version of the django-solo package does not include the change yet. What's up?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants