From ba75f8288d2863345025d6169da8a22e08d573a5 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Johannes=20M=C3=BCller?= <straightshoota@gmail.com>
Date: Sat, 18 Nov 2023 01:05:21 +0100
Subject: [PATCH] improvements

---
 _includes/site/breadcrumbs.html    | 12 +++++++-----
 _layouts/base.html                 |  2 +-
 _sass/components/_breadcrumbs.scss | 11 +++++++----
 3 files changed, 15 insertions(+), 10 deletions(-)

diff --git a/_includes/site/breadcrumbs.html b/_includes/site/breadcrumbs.html
index 26bc055e8..ef7f708d9 100644
--- a/_includes/site/breadcrumbs.html
+++ b/_includes/site/breadcrumbs.html
@@ -1,15 +1,17 @@
 <div class="section-nav">
   <nav class="breadcrumbs" aria-label="Breadcrumbs">
     {% capture separator %}<span aria-hidden="true">/</span>{% endcapture %}
+    {%- assign page_title = nil -%}
+    {%- assign page_nav = nil -%}
 
     <span class="breadcrumb">
       <a href="/" rel="index up">Home</a>
     </span>
 
-    {% if page.section -%}
-      {%- assign section = site.data.sections[page.section] -%}
+    {% if include.page.section -%}
+      {%- assign section = site.data.sections[include.page.section] -%}
       {% for item in section.breadcrumbs %}
-        {%- if item.url == page.url -%}
+        {%- if item.url == include.page.url -%}
           {% assign page_title = item.title %}
           {% assign page_nav = item.nav %}
           {% break %}
@@ -21,7 +23,7 @@
         {% if item.url -%}
           <a href="{{ item.url }}" rel="up">{{ item.title }}</a>
         {%- else -%}
-          {{ item.title }}
+          <span>{{ item.title }}</span>
         {%- endif %}
         </div>
       {% endfor %}
@@ -31,7 +33,7 @@
       {{ separator }}
 
       <span class="breadcrumb">
-        <a href="{{ page.url }}" aria-current="page">{{ page_title }}</a>
+        <a href="{{ include.page.url }}" aria-current="page">{{ page_title }}</a>
       </span>
     {% endif %}
   </nav>
diff --git a/_layouts/base.html b/_layouts/base.html
index 9a9aee547..df6427447 100644
--- a/_layouts/base.html
+++ b/_layouts/base.html
@@ -8,7 +8,7 @@
   {% include site/nav.html %}
 </header>
 
-{% include site/breadcrumbs.html %}
+{% include site/breadcrumbs.html page=page %}
 
 <main>
   {{ content }}
diff --git a/_sass/components/_breadcrumbs.scss b/_sass/components/_breadcrumbs.scss
index 0c6c33aed..4f67d4d1f 100644
--- a/_sass/components/_breadcrumbs.scss
+++ b/_sass/components/_breadcrumbs.scss
@@ -1,13 +1,16 @@
 .section-nav {
-  margin-block-start: var(--block-flow-sm);
   transition: 0.5s opacity;
   &:not(:hover) {
     opacity: 0.6;
   }
+
+  margin-block-start: var(--block-flow-xs);
+  &:first-child {
+    margin-block-start: var(--block-flow-sm);
+  }
 }
 
 .breadcrumbs {
-  position: relative;
   display: flex;
 
   --link-color: var(--light-text);
@@ -22,10 +25,10 @@
 .breadcrumb {
   display: inline flex;
 
-  > a {
+  > :first-child {
     padding-inline-end: var(--padding-xs);
   }
-  &:not(:first-child) > a {
+  &:not(:first-child) > :first-child {
     padding-inline-start: var(--padding-xs);
   }
 }