danvics/pediatric-ai-scribe-v3
1
0
Fork 0
Code Issues Pull requests Projects Releases 16 Packages Wiki Activity Actions

fix: the last browser confirm() in the resources library uses the app's confirm modal
All checks were successful
Forgejo Docker Build / Root app tests (push) Successful in 1m4s
Details
Forgejo Docker Build / Build Docker image (push) Successful in 19s
Details
Forgejo Docker Build / End-to-end (browser) (push) Successful in 9s
Details

Browse source 
Deleting a generated image asked with the browser's own confirm(), a bare
dialog that looks nothing like the rest of the app. It now goes through
showConfirm like every other confirmation, with the danger styling and a
Delete button.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015j4L7Eu5Jp8gVMPsXdPaZ2
This commit is contained in:
Daniel 2026-09-14 10:44:31 +02:00
parent 5d28ebad9c
commit 055c8d121b
1 changed files with 23 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
public/js/myResources.js
View file

@ -590,7 +590,10 @@
function deleteImage(image, tile, button) { function deleteImage(image, tile, button) {
// Asked, because it removes the picture from every deck that used it. The // Asked, because it removes the picture from every deck that used it. The
// deck keeps working; the figure is simply no longer there. // deck keeps working; the figure is simply no longer there.
if (!window.confirm('Delete this image? Any resource that used it will render without it.')) return; var message = 'Delete this image? Any resource that used it will render without it.';
if (!window.showConfirm) { if (window.confirm(message)) doDelete(); return; }
window.showConfirm(message, doDelete, { danger: true, confirmText: 'Delete' });
function doDelete() {
button.disabled = true; button.disabled = true;
fetch('/api/generated-images/' + encodeURIComponent(image.id), { fetch('/api/generated-images/' + encodeURIComponent(image.id), {
method: 'DELETE', headers: getAuthHeaders() method: 'DELETE', headers: getAuthHeaders()
@ -611,6 +614,7 @@
if (typeof showToast === 'function') showToast(err.message, 'error'); if (typeof showToast === 'function') showToast(err.message, 'error');
}); });
} }
}
// The library as last fetched. Held so searching and the Modify picker both // The library as last fetched. Held so searching and the Modify picker both
// work from one copy rather than each asking the server again. // work from one copy rather than each asking the server again.