Fixed breadcrumbs (#96)

* Fixed breadcrumbs Added breadcrumb chevron back in, and changed to show inside hero instead of content when one exists * Added check in case there is a hero
Bioconductor · Jul 31, 2023 · c7ef588 · c7ef588
1 parent e1d5b1e
commit c7ef588
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 6 deletions.
diff --git a/assets/images/icons/svgs/chevron-right-n400.svg b/assets/images/icons/svgs/chevron-right-n400.svg
diff --git a/assets/style/breadcrumbs.css b/assets/style/breadcrumbs.css
@@ -5,10 +5,11 @@
   margin: 0;
   padding: 0;
   line-height: 1.3rem;
+  color: var(--neutral-n400);
 }
 
 .breadcrumbs li:not(:first-of-type) {
-  background-image: url("/images/chevron_right.svg");
+  background-image: url("/images/icons/svgs/chevron-right-n400.svg");
   background-repeat: no-repeat;
   padding-left: 1.625rem; /* make space for the background image */
 }

diff --git a/layouts/default.html b/layouts/default.html
@@ -7,10 +7,16 @@
 >
   <%= render("/_sitehead/") %>
   <body>
-    <%= render("/components/header/") %> <%= render("/_nav/",
-    :section_navigation => has_subnav?(@item) && subnav_items(@item).length > 0)
-    %> <% if @item[:hero] %>
-    <section class="hero"><%= render(@item[:hero]) %></section>
+    <%= render("/components/header/") %>
+    <%= render("/_nav/", :section_navigation => has_subnav?(@item) && subnav_items(@item).length > 0) %>
+
+    <% if @item[:hero] %>
+    <section class="hero">
+      <% if (@item.identifier != '/') %>
+        <%= render("/components/breadcrumbs/") %>
+      <% end %>
+      <%= render(@item[:hero]) %>
+    </section>
     <%end%>
 
     <main>
@@ -29,7 +35,7 @@
           <% end %>
 
           <section class="content">
-            <% if (@item.identifier != '/') %>
+            <% if (@item.identifier != '/' && !@item[:hero]) %>
               <%= render("/components/breadcrumbs/") %>
             <% end %>