sidebar.logo is now normalized as an object with light and dark properties

So `sidebar.logo` is never undefined - it may have light or dark undefined, but the object itself exists.

Author: cderv

src/resources/projects/website/templates/sidebar.ejs

@@ -19,11 +19,13 @@
     // may be displayed in different locations, so this will determine where to display the tools
     // "logo", "title", "search", "fallthrough"
 
+    let hasLogo = sidebar.logo.light || sidebar.logo.dark;
+
     let toolsLocation;
     // Under the title if that will be displayed
     if (sidebar.title && !navbar) {
       toolsLocation = "title";
-    } else if (sidebar.logo) {
+    } else if (sidebar.logo.light || sidebar.logo.dark) {
       toolsLocation = "logo";
     } else if (sidebar.search && sidebar.search !== "overlay") {
       toolsLocation = "search";
@@ -32,9 +34,9 @@
     }
   %>
     
-  <% if (sidebar.logo || (sidebar.title && !navbar)) { %>
-    <div class="pt-lg-2 mt-2 <%= alignCss %> sidebar-header<%= sidebar.logo && sidebar.title ? ' sidebar-header-stacked' : '' %>">
-    <% if (sidebar.logo) { %>
+  <% if (hasLogo || (sidebar.title && !navbar)) { %>
+    <div class="pt-lg-2 mt-2 <%= alignCss %> sidebar-header<%= hasLogo && sidebar.title ? ' sidebar-header-stacked' : '' %>">
+    <% if (hasLogo) { %>
       <a href="<%- sidebar['logo-href'] || '/index.html' %>" class="sidebar-logo-link">
       <% if (sidebar.logo.light) { %>
       <img src="<%- sidebar.logo.light.path %>" alt="<%- sidebar.logo.light.alt || '' %>" class="sidebar-logo light-content py-0 d-lg-inline d-none"/>