From 16450619325f7b7a8af08ae6618866c86c2a87df Mon Sep 17 00:00:00 2001
From: James <james@jongsma.me>
Date: Sat, 14 Mar 2026 23:39:17 -0400
Subject: [PATCH] =?UTF-8?q?fix:=20menu=20buttons=20broken=20when=20title?=
 =?UTF-8?q?=20contains=20apostrophes=20=E2=80=94=20use=20showAttachModalBy?=
 =?UTF-8?q?Id=20to=20avoid=20inline=20onclick=20escaping=20issues?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 portal/templates/app/project.html | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/portal/templates/app/project.html b/portal/templates/app/project.html
index 8a51b77..0382136 100644
--- a/portal/templates/app/project.html
+++ b/portal/templates/app/project.html
@@ -526,8 +526,8 @@
             <td>${labelPill(label)}</td>
             <td style="text-align:center">
               ${item.answer_count > 0
-                ? `<span class="badge" style="background:rgba(74,222,128,.15);color:#4ade80;cursor:pointer" onclick="showAttachModal('${eid}','${escHtml(title)}')">${item.answer_count}</span>`
-                : `<span style="color:var(--ds-tx3);cursor:pointer;font-size:12px" onclick="showAttachModal('${eid}','${escHtml(title)}')">&mdash;</span>`}
+                ? `<span class="badge" style="background:rgba(74,222,128,.15);color:#4ade80;cursor:pointer" onclick="showAttachModalById('${eid}')">${item.answer_count}</span>`
+                : `<span style="color:var(--ds-tx3);cursor:pointer;font-size:12px" onclick="showAttachModalById('${eid}')">&mdash;</span>`}
             </td>
             <td style="max-width:140px;overflow:hidden;text-overflow:ellipsis;color:var(--ds-tx2);font-size:12px" title="${escHtml(buyerComment)}">${escHtml(buyerComment)}</td>
             <td style="max-width:140px;overflow:hidden;text-overflow:ellipsis;color:var(--ds-tx2);font-size:12px" title="${escHtml(sellerComment)}">${escHtml(sellerComment)}</td>
@@ -535,7 +535,7 @@
             <td style="position:relative">
               <button onclick="toggleMenu(event,'${eid}')" style="background:none;border:none;cursor:pointer;color:var(--ds-tx3);font-size:16px;padding:2px 6px">&hellip;</button>
               <div id="menu-${eid}" class="actions-menu" style="display:none">
-                <button onclick="showAttachModal('${eid}','${escHtml(title)}')">Attach Answer</button>
+                <button onclick="showAttachModalById('${eid}')">Attach Answer</button>
                 <button onclick="editRequest('${eid}', event)">Edit</button>
                 <button onclick="deleteEntry('${eid}')">Delete</button>
               </div>
@@ -800,6 +800,13 @@
     }
 
     // ---- Attach answer modal ----
+    function showAttachModalById(reqId) {
+      const item = treeData.find(t => t.entry_id === reqId);
+      const d = item ? (item.data || {}) : {};
+      const title = d.title || d.description || 'Untitled';
+      showAttachModal(reqId, title);
+    }
+
     function showAttachModal(reqId, reqTitle) {
       attachRequestId = reqId;
       document.getElementById('attachReqTitle').textContent = reqTitle;