fix: EJS syntax error in matrix body — replace inline onclick with helper functions

This commit is contained in:
Claude Chronicler #88
2026-04-14 15:56:45 +00:00
parent 38edf84da2
commit d77e35a4a1
2 changed files with 14 additions and 5 deletions

View File

@@ -203,7 +203,7 @@
<%= (config && config.current_version) ? config.current_version : 'Not set' %>
</span>
<button id="version-edit-btn-<%= server.identifier %>"
onclick="document.getElementById('version-form-<%= server.identifier %>').style.display='block';this.style.display='none'"
onclick="toggleVersionForm('<%= server.identifier %>')"
style="font-size:10px;background:#333;border:1px solid #555;color:#aaa;padding:2px 8px;border-radius:4px;cursor:pointer;">
✏️ Edit
</button>
@@ -215,7 +215,7 @@
style="font-size:12px;background:#1a1a1a;border:1px solid #555;color:#e0e0e0;padding:4px 8px;border-radius:4px;width:140px;" />
<button onclick="saveVersion('<%= server.identifier %>')"
style="font-size:11px;background:#2563eb;color:#fff;border:none;padding:4px 10px;border-radius:4px;cursor:pointer;">Save</button>
<button onclick="document.getElementById('version-form-<%= server.identifier %>').style.display='none';document.getElementById('version-edit-btn-<%= server.identifier %>').style.display='inline'"
<button onclick="hideVersionForm('<%= server.identifier %>')"
style="font-size:11px;background:#333;border:1px solid #555;color:#aaa;padding:4px 8px;border-radius:4px;cursor:pointer;">Cancel</button>
</div>
<div id="version-result-<%= server.identifier %>" style="margin-top:4px;font-size:11px;"></div>
@@ -437,7 +437,7 @@
<%= (config && config.current_version) ? config.current_version : 'Not set' %>
</span>
<button id="version-edit-btn-<%= server.identifier %>"
onclick="document.getElementById('version-form-<%= server.identifier %>').style.display='block';this.style.display='none'"
onclick="toggleVersionForm('<%= server.identifier %>')"
style="font-size:10px;background:#333;border:1px solid #555;color:#aaa;padding:2px 8px;border-radius:4px;cursor:pointer;">
✏️ Edit
</button>
@@ -449,7 +449,7 @@
style="font-size:12px;background:#1a1a1a;border:1px solid #555;color:#e0e0e0;padding:4px 8px;border-radius:4px;width:140px;" />
<button onclick="saveVersion('<%= server.identifier %>')"
style="font-size:11px;background:#2563eb;color:#fff;border:none;padding:4px 10px;border-radius:4px;cursor:pointer;">Save</button>
<button onclick="document.getElementById('version-form-<%= server.identifier %>').style.display='none';document.getElementById('version-edit-btn-<%= server.identifier %>').style.display='inline'"
<button onclick="hideVersionForm('<%= server.identifier %>')"
style="font-size:11px;background:#333;border:1px solid #555;color:#aaa;padding:4px 8px;border-radius:4px;cursor:pointer;">Cancel</button>
</div>
<div id="version-result-<%= server.identifier %>" style="margin-top:4px;font-size:11px;"></div>

View File

@@ -55,7 +55,6 @@ async function saveVersion(identifier) {
const text = await res.text();
result.innerHTML = text;
// Update the display
if (res.ok) {
const display = document.getElementById('version-display-' + identifier);
const span = display.querySelector('span');
@@ -66,4 +65,14 @@ async function saveVersion(identifier) {
result.innerHTML = '<span style="color:#ef4444">❌ Failed to save</span>';
}
}
function toggleVersionForm(identifier) {
document.getElementById('version-form-' + identifier).style.display = 'block';
document.getElementById('version-edit-btn-' + identifier).style.display = 'none';
}
function hideVersionForm(identifier) {
document.getElementById('version-form-' + identifier).style.display = 'none';
document.getElementById('version-edit-btn-' + identifier).style.display = 'inline';
}
</script>