diff --git a/js/docsView.js b/js/docsView.js index 7c0366ec..59a37fc9 100644 --- a/js/docsView.js +++ b/js/docsView.js @@ -105,21 +105,35 @@ function buildLink(item) { return link; } +function getChevronIcon(expanded) { + if (expanded) { + // Chevron Down + return ` + + `; + } + // Chevron Right + return ` + + `; +} + function buildToggleButton(item, controlsId, expanded, groupList) { const button = document.createElement("button"); button.type = "button"; - button.className = "btn-ghost btn-xs"; + button.className = "btn-ghost btn-xs btn-square text-muted hover:text-text-primary"; button.setAttribute("aria-expanded", expanded ? "true" : "false"); button.setAttribute("aria-controls", controlsId); button.setAttribute("aria-label", `Toggle ${item.title || item.slug} section`); - button.textContent = expanded ? "Hide" : "Show"; + button.innerHTML = getChevronIcon(expanded); button.addEventListener("click", () => { const isExpanded = button.getAttribute("aria-expanded") === "true"; const nextExpanded = !isExpanded; button.setAttribute("aria-expanded", nextExpanded ? "true" : "false"); - button.textContent = nextExpanded ? "Hide" : "Show"; + button.innerHTML = getChevronIcon(nextExpanded); if (groupList) { groupList.hidden = !nextExpanded; + groupList.classList.toggle("hidden", !nextExpanded); } }); return button; @@ -152,6 +166,9 @@ function renderTocItems(items, container, level = 0, groupPrefix = "docs-toc") { childList.id = groupId; childList.classList.add("ml-md", "border-l", "border-border", "pl-md"); childList.hidden = !expanded; + if (!expanded) { + childList.classList.add("hidden"); + } const toggle = buildToggleButton(item, groupId, expanded, childList); row.appendChild(toggle); @@ -220,8 +237,9 @@ function updateActiveToc(slug) { groups.forEach((group) => { if (group?.button && group?.list) { group.button.setAttribute("aria-expanded", "true"); - group.button.textContent = "Hide"; + group.button.innerHTML = getChevronIcon(true); group.list.hidden = false; + group.list.classList.remove("hidden"); } }); } diff --git a/package-lock.json b/package-lock.json index ec60e7f2..20c889e1 100644 --- a/package-lock.json +++ b/package-lock.json @@ -17,6 +17,7 @@ "ws": "^8.19.0" }, "devDependencies": { + "@noble/hashes": "^2.0.1", "@playwright/test": "^1.58.0", "@tailwindcss/typography": "^0.5.15", "autoprefixer": "^10.4.23", diff --git a/package.json b/package.json index 37396b9b..aa0a146c 100644 --- a/package.json +++ b/package.json @@ -32,6 +32,7 @@ "telemetry:aggregate": "node scripts/agent/telemetry-aggregator.mjs" }, "devDependencies": { + "@noble/hashes": "^2.0.1", "@playwright/test": "^1.58.0", "@tailwindcss/typography": "^0.5.15", "autoprefixer": "^10.4.23", diff --git a/tests/ui/video-list-view-popular-tags.test.mjs b/tests/ui/video-list-view-popular-tags.test.mjs index 5d07834e..e2f98a68 100644 --- a/tests/ui/video-list-view-popular-tags.test.mjs +++ b/tests/ui/video-list-view-popular-tags.test.mjs @@ -76,7 +76,7 @@ test("VideoListView renders sorted popular tag pills", () => { assert.equal(buttons.length, 3); assert.deepEqual( Array.from(buttons, (button) => button.dataset.tag), - ["Nostr", "Video", "Learning"], + ["#Nostr", "#Video", "#Learning"], ); }); @@ -90,7 +90,7 @@ test("VideoListView applies tag preference variants to popular tags", () => { view.mount(listRoot); view.setPopularTagsContainer(tagsRoot); view.setTagPreferenceStateResolver((tag) => - tag === "Nostr" ? "interest" : tag === "Video" ? "disinterest" : "neutral", + tag === "#Nostr" ? "interest" : tag === "#Video" ? "disinterest" : "neutral", ); const videos = [ diff --git a/views/docs.html b/views/docs.html index 8ed33524..21197b7a 100644 --- a/views/docs.html +++ b/views/docs.html @@ -19,12 +19,26 @@