From e9bcb1b6645a36e5358af645e0b84c7d97a30788 Mon Sep 17 00:00:00 2001 From: Knut Forkalsrud Date: Sat, 3 Jan 2026 15:36:17 -0800 Subject: [PATCH] Cut back on string concatenation when building up the DOM --- src/main/webapp/assets/render.js | 24 ++++++++++++++++++------ 1 file changed, 18 insertions(+), 6 deletions(-) diff --git a/src/main/webapp/assets/render.js b/src/main/webapp/assets/render.js index 222839f..11f2f10 100644 --- a/src/main/webapp/assets/render.js +++ b/src/main/webapp/assets/render.js @@ -91,13 +91,25 @@ $(function() { href = "/photo/album" + encUri(entry.path) + "?size=" + picSize; poster = "/photo/album" + encUri(entry.path); } - var gridDiv = $("
\n" - + " " + entry.name + "
\n" - + " \n" - + "
\n"); + var imgTag = $(""); + imgTag.addClass(entry.type + "pic") + .attr("src", poster + "?size=" + thmb) + .attr("width", dim.w) + .attr("height", dim.h) + .attr("border", "0"); + var anchor = $(""); + anchor.addClass("ss") + .attr("id", "ent" + idx) + .attr("href", href) + .attr("title", entry.name) + anchor.append(imgTag); - var captionP = $("

\n"); + + var gridDiv = $("
").addClass("grid"); + + gridDiv.append($("").addClass("name").text(entry.name), "
\n", $("
").append(anchor)) + + var captionP = $("

").addClass("caption"); captionP.attr("id", entry.name); if (entry.caption) { captionP.text(entry.caption);