From 37873e02b07aa1e77f6c8faeb9d8a999a3a047f2 Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Thu, 9 Jan 2025 10:07:03 -0800
Subject: [PATCH] Better breadcrumbs on database and table page, closes #2454

---
 datasette/templates/database.html |  4 ++++
 datasette/templates/table.html    |  2 +-
 tests/test_html.py                | 19 +++++++------------
 3 files changed, 12 insertions(+), 13 deletions(-)

diff --git a/datasette/templates/database.html b/datasette/templates/database.html
index 0898559a..66f288dc 100644
--- a/datasette/templates/database.html
+++ b/datasette/templates/database.html
@@ -9,6 +9,10 @@
 
 {% block body_class %}db db-{{ database|to_css_class }}{% endblock %}
 
+{% block crumbs %}
+{{ crumbs.nav(request=request, database=database) }}
+{% endblock %}
+
 {% block content %}
 <div class="page-header" style="border-color: #{{ database_color }}">
     <h1>{{ metadata.title or database }}{% if private %} 🔒{% endif %}</h1>
diff --git a/datasette/templates/table.html b/datasette/templates/table.html
index c9e0e87b..25ff31ef 100644
--- a/datasette/templates/table.html
+++ b/datasette/templates/table.html
@@ -17,7 +17,7 @@
 {% block body_class %}table db-{{ database|to_css_class }} table-{{ table|to_css_class }}{% endblock %}
 
 {% block crumbs %}
-{{ crumbs.nav(request=request, database=database) }}
+{{ crumbs.nav(request=request, database=database, table=table) }}
 {% endblock %}
 
 {% block content %}
diff --git a/tests/test_html.py b/tests/test_html.py
index 4d95a8fa..3fd5f1f5 100644
--- a/tests/test_html.py
+++ b/tests/test_html.py
@@ -1089,12 +1089,16 @@ async def test_redirect_percent_encoding_to_tilde_encoding(ds_client, path, expe
 @pytest.mark.parametrize(
     "path,config,expected_links",
     (
-        ("/fixtures", {}, [("/", "home")]),
-        ("/fixtures", {"allow": False, "databases": {"fixtures": {"allow": True}}}, []),
+        ("/fixtures", {}, [("/", "home"), ("/fixtures", "fixtures")]),
+        (
+            "/fixtures",
+            {"allow": False, "databases": {"fixtures": {"allow": True}}},
+            [("/fixtures", "fixtures")],
+        ),
         (
             "/fixtures/facetable",
             {"allow": False, "databases": {"fixtures": {"allow": True}}},
-            [("/fixtures", "fixtures")],
+            [("/fixtures", "fixtures"), ("/fixtures/facetable", "facetable")],
         ),
         (
             "/fixtures/facetable/1",
@@ -1110,15 +1114,6 @@ async def test_redirect_percent_encoding_to_tilde_encoding(ds_client, path, expe
             {"allow": False, "databases": {"fixtures": {"allow": True}}},
             [("/fixtures", "fixtures"), ("/fixtures/facetable", "facetable")],
         ),
-        # TODO: what
-        # (
-        #    "/fixtures/facetable/1",
-        #    {
-        #        "allow": False,
-        #        "databases": {"fixtures": {"tables": {"facetable": {"allow": True}}}},
-        #    },
-        #    [("/fixtures/facetable", "facetable")],
-        # ),
     ),
 )
 async def test_breadcrumbs_respect_permissions(ds_client, path, config, expected_links):