fix: object history breadcrumb title (#1489)

Author: lukasvinclav

src/unfold/contrib/simple_history/templates/simple_history/object_history_form.html

@@ -1,6 +1,6 @@
 {% extends "admin/change_form.html" %}
 
-{% load admin_urls i18n url from simple_history_compat %}
+{% load i18n %}
 
 {% block submit_buttons_top %}
     {% include "simple_history/submit_line.html" %}

src/unfold/contrib/simple_history/templates/simple_history/object_history_list.html

@@ -1,5 +1,4 @@
 {% load i18n %}
-{% load url from simple_history_compat %}
 {% load admin_urls %}
 {% load getattribute from getattributes %}
 

src/unfold/templatetags/unfold.py

@@ -610,35 +610,45 @@ def querystring_params(
 @register.simple_tag(takes_context=True)
 def header_title(context: RequestContext) -> str:
     parts = []
+    opts = context.get("opts")
 
-    if opts := context.get("opts"):
+    if opts:
         parts.append(
             {
                 "link": reverse_lazy("admin:app_list", args=[opts.app_label]),
                 "title": opts.app_config.verbose_name,
             }
         )
 
-        parts.append(
-            {
-                "link": reverse_lazy(
-                    f"admin:{opts.app_label}_{opts.model_name}_changelist"
-                ),
-                "title": opts.verbose_name_plural,
-            }
-        )
-
         if (original := context.get("original")) and not isinstance(original, str):
             parts.append(
                 {
                     "link": reverse_lazy(
-                        f"admin:{opts.app_label}_{opts.model_name}_change",
+                        f"admin:{original._meta.app_label}_{original._meta.model_name}_changelist"
+                    ),
+                    "title": original._meta.verbose_name_plural,
+                }
+            )
+
+            parts.append(
+                {
+                    "link": reverse_lazy(
+                        f"admin:{original._meta.app_label}_{original._meta.model_name}_change",
                         args=[original.pk],
                     ),
                     "title": original,
                 }
             )
         elif object := context.get("object"):
+            parts.append(
+                {
+                    "link": reverse_lazy(
+                        f"admin:{object._meta.app_label}_{object._meta.model_name}_changelist"
+                    ),
+                    "title": object._meta.verbose_name_plural,
+                }
+            )
+
             parts.append(
                 {
                     "link": reverse_lazy(
@@ -648,6 +658,15 @@ def header_title(context: RequestContext) -> str:
                     "title": object,
                 }
             )
+        else:
+            parts.append(
+                {
+                    "link": reverse_lazy(
+                        f"admin:{opts.app_label}_{opts.model_name}_changelist"
+                    ),
+                    "title": opts.verbose_name_plural,
+                }
+            )
     elif object := context.get("object"):
         parts.append(
             {
@@ -689,7 +708,7 @@ def header_title(context: RequestContext) -> str:
                 "link": reverse_lazy(
                     f"admin:{model_admin.model._meta.app_label}_{model_admin.model._meta.model_name}_changelist",
                 ),
-                "title": model_admin.model._meta.verbose_name,
+                "title": model_admin.model._meta.verbose_name_plural,
             }
         )
 
@@ -704,7 +723,7 @@ def header_title(context: RequestContext) -> str:
         username = (
             context.request.user.get_short_name() or context.request.user.get_username()
         )
-        parts.append({"title": f"{_('Welcome')} <strong>{username}</strong>"})
+        parts.append({"title": f"{_('Welcome')} {username}"})
 
     return render_to_string(
         "unfold/helpers/header_title.html",