adjust comments

This commit is contained in:
Ryan
2025-04-07 23:26:37 -04:00
committed by GitHub
parent 9701747214
commit 82eadebe3b

View File

@@ -58,6 +58,9 @@ function getParentFolder(folder) {
return lastSlash === -1 ? "root" : folder.substring(0, lastSlash); return lastSlash === -1 ? "root" : folder.substring(0, lastSlash);
} }
/* ----------------------
Breadcrumb Functions
----------------------*/
function renderBreadcrumb(normalizedFolder) { function renderBreadcrumb(normalizedFolder) {
if (!normalizedFolder || normalizedFolder === "") return ""; if (!normalizedFolder || normalizedFolder === "") return "";
@@ -75,14 +78,14 @@ function renderBreadcrumb(normalizedFolder) {
} }
// --- NEW: Breadcrumb Delegation Setup --- // --- NEW: Breadcrumb Delegation Setup ---
// bindBreadcrumbEvents(); removed in favor of delegation
export function setupBreadcrumbDelegation() { export function setupBreadcrumbDelegation() {
const container = document.getElementById("fileListTitle"); const container = document.getElementById("fileListTitle");
if (!container) { if (!container) {
console.error("Breadcrumb container (fileListTitle) not found."); console.error("Breadcrumb container (fileListTitle) not found.");
return; return;
} }
// Remove any existing event listeners to avoid duplicates.
// Remove any previous delegation listeners if necessary
container.removeEventListener("click", breadcrumbClickHandler); container.removeEventListener("click", breadcrumbClickHandler);
container.removeEventListener("dragover", breadcrumbDragOverHandler); container.removeEventListener("dragover", breadcrumbDragOverHandler);
container.removeEventListener("dragleave", breadcrumbDragLeaveHandler); container.removeEventListener("dragleave", breadcrumbDragLeaveHandler);
@@ -97,19 +100,16 @@ export function setupBreadcrumbDelegation() {
// Click handler via delegation // Click handler via delegation
function breadcrumbClickHandler(e) { function breadcrumbClickHandler(e) {
// Look for the nearest ancestor with the "breadcrumb-link" class.
const link = e.target.closest(".breadcrumb-link"); const link = e.target.closest(".breadcrumb-link");
if (!link) return; if (!link) return;
e.stopPropagation(); e.stopPropagation();
e.preventDefault(); // Prevent default link behavior if needed e.preventDefault();
const folder = link.getAttribute("data-folder"); const folder = link.getAttribute("data-folder");
window.currentFolder = folder; window.currentFolder = folder;
localStorage.setItem("lastOpenedFolder", folder); localStorage.setItem("lastOpenedFolder", folder);
// Update the container with sanitized breadcrumbs. // Update the container with sanitized breadcrumbs.
// (If you prefer, you can render the breadcrumbs into a dedicated container
// and update the title separately.)
const container = document.getElementById("fileListTitle"); const container = document.getElementById("fileListTitle");
const sanitizedBreadcrumb = DOMPurify.sanitize(renderBreadcrumb(folder)); const sanitizedBreadcrumb = DOMPurify.sanitize(renderBreadcrumb(folder));
container.innerHTML = "Files in (" + sanitizedBreadcrumb + ")"; container.innerHTML = "Files in (" + sanitizedBreadcrumb + ")";