Skip to content

Commit 9426caa

Browse files
committed
Merge branch 'main' of github.com:quarto-dev/quarto-cli into main
2 parents f9dfada + c889a73 commit 9426caa

File tree

4 files changed

+35
-17
lines changed

4 files changed

+35
-17
lines changed

src/core/path.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -195,12 +195,14 @@ export function resolveGlobs(
195195
return false;
196196
} else if (options?.mode === "always") {
197197
return true;
198-
} else {
198+
} else if (options?.mode === "auto") {
199199
if (preferSmart) {
200200
return true;
201201
} else {
202202
return isGlob(glob);
203203
}
204+
} else {
205+
return true;
204206
}
205207
};
206208
const smartGlob = useSmartGlobs();

src/project/project-config.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,10 @@ export function normalizeSidebarItem(
5959
// case
6060
item.sectionId = `${kQuartoSidebarPrefix}${context.counter}`;
6161
delete item.section;
62+
63+
// If this is a section, we should insist that it have 'contents'
64+
// even if they are empty.
65+
item.contents = item.contents || [];
6266
}
6367

6468
// handle subitems

src/resources/formats/html/templates/quarto-html.ejs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@ window.document.addEventListener("DOMContentLoaded", function (event) {
1717
}
1818
1919
const toggleBodyColorPrimary = () => {
20-
toggleBodyColorMode(window.document.querySelector("link#quarto-bootstrap"));
20+
const bsSheetEl = window.document.querySelector("link#quarto-bootstrap");
21+
if (bsSheetEl) {
22+
toggleBodyColorMode(bsSheetEl);
23+
}
2124
}
2225
2326
toggleBodyColorPrimary();

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

Lines changed: 24 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -14,22 +14,31 @@
1414
<% isCollapsed = collapse <= depth && !item.expanded %>
1515
1616
<li class="sidebar-item sidebar-item-section">
17-
<div class="sidebar-item-container">
18-
<% if (item.href) { %>
19-
<a href="<%- item.href %>" class="sidebar-item-text sidebar-link<%- item.active ? " active" : "" %>"><%= item.text %></a>
20-
<% } else { %>
21-
<a class="sidebar-item-text sidebar-link text-start<%- isCollapsed ? " collapsed" : "" %>" data-bs-toggle="collapse" data-bs-target="#<%- sectionId %>" aria-expanded="<%- isCollapsed ? "false" : "true" %>"><%= item.text %></a>
22-
<% } %>
23-
<a class="sidebar-item-toggle text-start<%- isCollapsed ? " collapsed" : "" %>" data-bs-toggle="collapse" data-bs-target="#<%- sectionId %>" aria-expanded="<%- isCollapsed ? "false" : "true" %>">
24-
<i class="bi bi-chevron-right ms-2"></i>
25-
</a>
26-
</div>
27-
<ul id="<%- sectionId %>" class="collapse list-unstyled sidebar-section depth<%-depth%> <%- isCollapsed ? "" : "show" %>">
28-
<% item.contents.forEach(subItem => { %>
29-
<% partial('sidebaritem.ejs', { item: subItem, depth: depth + 1, collapse: collapse, borderColor: borderColor }) %>
30-
<% }) %>
31-
</ul>
17+
<% if (item.contents.length > 0) { %>
18+
<div class="sidebar-item-container">
19+
<% if (item.href) { %>
20+
<a href="<%- item.href %>" class="sidebar-item-text sidebar-link<%- item.active ? " active" : "" %>"><%= item.text %></a>
21+
<% } else { %>
22+
<a class="sidebar-item-text sidebar-link text-start<%- isCollapsed ? " collapsed" : "" %>" data-bs-toggle="collapse" data-bs-target="#<%- sectionId %>" aria-expanded="<%- isCollapsed ? "false" : "true" %>"><%= item.text %></a>
23+
<% } %>
24+
<a class="sidebar-item-toggle text-start<%- isCollapsed ? " collapsed" : "" %>" data-bs-toggle="collapse" data-bs-target="#<%- sectionId %>" aria-expanded="<%- isCollapsed ? "false" : "true" %>">
25+
<i class="bi bi-chevron-right ms-2"></i>
26+
</a>
27+
</div>
28+
<ul id="<%- sectionId %>" class="collapse list-unstyled sidebar-section depth<%-depth%> <%- isCollapsed ? "" : "show" %>">
29+
<% item.contents.forEach(subItem => { %>
30+
<% partial('sidebaritem.ejs', { item: subItem, depth: depth + 1, collapse: collapse, borderColor: borderColor }) %>
31+
<% }) %>
32+
</ul>
33+
<% } else { %>
34+
<span class="sidebar-item-text sidebar-link text-start"><%= item.text %></span>
35+
<% } %>
36+
3237
</li>
3338
<% } else if (item.text && item.text.match(/^\-+$/)) { %>
3439
<li class="px-0"><hr class="sidebar-divider hi <%- borderColor %>"></li>
40+
<% } else if (item.text) { %>
41+
<li class="sidebar-item">
42+
<%= item.text %>
43+
</li>
3544
<% } %>

0 commit comments

Comments
 (0)