Skip to content

asset-validation: view/edit feedback note without toggling rating #39

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thibaut-bp merged 1 commit into from
Jun 24, 2026
Merged

asset-validation: view/edit feedback note without toggling rating #39

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 15 additions & 2 deletions projects/asset-validation/index.html
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1103,6 +1103,8 @@
.tile-rate button.active.like { background: var(--success); }
.tile-rate button.active.dislike { background: var(--error); }
.tile-rate button:hover { background: var(--ink); }
.tile-rate button.note.has-note { background: var(--amber); color: var(--ink); }
.lb-note.has-note { color: var(--amber); border-color: var(--amber); }
.tile-pending {
aspect-ratio: 1;
background: var(--card);
Expand Down Expand Up @@ -1475,6 +1477,7 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
<div class="lb-actions">
<button class="lb-btn lb-dislike" onclick="event.stopPropagation(); rateCurrent('dislike')" title="Dislike (D)">👎 Dislike</button>
<button class="lb-btn lb-like" onclick="event.stopPropagation(); rateCurrent('like')" title="Like (L)">👍 Like</button>
<button class="lb-btn lb-note" onclick="event.stopPropagation(); editNoteCurrent()" title="View / edit your note (N)">✎ Note</button>
</div>
</div>

Expand Down Expand Up @@ -2466,6 +2469,7 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
<div class="tile-rate">
<button class="dislike ${rating === "dislike" ? "active" : ""}" onclick="event.stopPropagation(); rate('${r.id}','dislike')" title="Dislike">👎</button>
<button class="like ${rating === "like" ? "active" : ""}" onclick="event.stopPropagation(); rate('${r.id}','like')" title="Like">👍</button>
<button class="note ${FEEDBACK[r.id] ? "has-note" : ""}" onclick="event.stopPropagation(); editNote('${r.id}')" title="${FEEDBACK[r.id] ? "Edit your note" : "Add a note"}">✎</button>
</div>
</div>`;
}
Expand Down Expand Up @@ -2510,6 +2514,7 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
renderScoreSummary();
if (LIGHTBOX_IDX != null) updateLightboxRateButtons();
// Prompt for free-text feedback on both like and dislike (skippable)
FEEDBACK_EDIT = false;
openFeedbackModal(id, value);
}

Expand Down Expand Up @@ -2548,6 +2553,9 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
}

let FEEDBACK_TARGET = null;
let FEEDBACK_EDIT = false; // true when re-opening a note to view/edit (no rating change, no auto-advance)
function editNote(id) { FEEDBACK_EDIT = true; openFeedbackModal(id, LIKES[id]); }
function editNoteCurrent() { const r = RESULTS[LIGHTBOX_IDX]; if (r) editNote(r.id); }
function openFeedbackModal(resultId, rating) {
FEEDBACK_TARGET = resultId;
const overlay = document.getElementById("feedbackModal");
Expand Down Expand Up @@ -2592,13 +2600,15 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
closeFeedbackModal();
renderResults();
renderScoreSummary();
advanceReview(ratedId);
if (!FEEDBACK_EDIT) advanceReview(ratedId);
FEEDBACK_EDIT = false;
maybeOpenConfirmSubmitModal();
}
function skipCurrentFeedback() {
const ratedId = FEEDBACK_TARGET;
closeFeedbackModal();
advanceReview(ratedId);
if (!FEEDBACK_EDIT) advanceReview(ratedId);
FEEDBACK_EDIT = false;
maybeOpenConfirmSubmitModal();
}

Expand Down Expand Up @@ -2857,6 +2867,8 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
const rating = LIKES[r.id] || "";
document.querySelector(".lb-like").classList.toggle("active", rating === "like");
document.querySelector(".lb-dislike").classList.toggle("active", rating === "dislike");
const nb = document.querySelector(".lb-note");
if (nb) nb.classList.toggle("has-note", !!FEEDBACK[r.id]);
}
function rateCurrent(value) {
const r = RESULTS[LIGHTBOX_IDX];
Expand All @@ -2876,6 +2888,7 @@ <h3 style="margin:0 0 6px; font-family:'Outfit',sans-serif; font-weight:700; fon
else if (e.key === "ArrowRight") lightboxNav(1);
else if (e.key.toLowerCase() === "l") rateCurrent("like");
else if (e.key.toLowerCase() === "d") rateCurrent("dislike");
else if (e.key.toLowerCase() === "n") editNoteCurrent();
});


Expand Down
Loading