bvandeusen/imagerepo
Archived
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor(js): use display_name from API instead of client-side split

Browse Source 
Every user-facing tag rendering now reads tag.display_name from the
server JSON. The client-side split-on-colon fallback is removed.
Series-info link switches from ?tag=<name> to ?tag_id=<int>.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
This commit is contained in:
Bryan Van Deusen
2026-04-22 16:16:38 -04:00
parent 54c3d96e45
commit c9fce48a39
2 changed files with 5 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
app/static/js/tag-editor.js
+2 -3
View File
@@ -52,9 +52,8 @@ document.addEventListener('DOMContentLoaded', () => {
} }
fandomAutocomplete.innerHTML = tags.map((t, i) => { fandomAutocomplete.innerHTML = tags.map((t, i) => {
const displayName = t.name.includes(':') ? t.name.split(':')[1] : t.name; return `<div class="tag-autocomplete-item" data-index="${i}" data-id="${t.id}" data-name="${t.display_name}">
return `<div class="tag-autocomplete-item" data-index="${i}" data-id="${t.id}" data-name="${displayName}"> <span class="tag-autocomplete-name">🎭 ${t.display_name}</span>
<span class="tag-autocomplete-name">🎭 ${displayName}</span>
</div>`; </div>`;
}).join(''); }).join('');
app/static/js/view-modal.js
+3 -8
View File
@@ -457,8 +457,7 @@ document.addEventListener('DOMContentLoaded', () => {
seriesTags.forEach(tag => { seriesTags.forEach(tag => {
const opt = document.createElement('option'); const opt = document.createElement('option');
opt.value = tag.id; opt.value = tag.id;
// Display name without prefix opt.textContent = tag.display_name;
opt.textContent = tag.name.includes(':') ? tag.name.split(':', 2)[1] : tag.name;
seriesSelect.appendChild(opt); seriesSelect.appendChild(opt);
}); });
@@ -514,13 +513,9 @@ document.addEventListener('DOMContentLoaded', () => {
if (!seriesInfoEl || !seriesPage) return; if (!seriesInfoEl || !seriesPage) return;
currentSeriesPageId = seriesPage.id; currentSeriesPageId = seriesPage.id;
// Display series name (strip prefix)
const displayName = seriesPage.series_name.includes(':')
? seriesPage.series_name.split(':', 2)[1]
: seriesPage.series_name;
if (seriesNameEl) { if (seriesNameEl) {
seriesNameEl.textContent = displayName; seriesNameEl.textContent = seriesPage.series_name;
seriesNameEl.href = `/gallery?tag=${encodeURIComponent(seriesPage.series_name)}`; seriesNameEl.href = `/gallery?tag_id=${encodeURIComponent(seriesPage.series_tag_id)}`;
} }
// Display page number // Display page number