From ff8162922bbdc14c96eeff35819fc7e36975cf53 Mon Sep 17 00:00:00 2001 From: Timothy Farrell Date: Wed, 3 Jan 2018 16:36:09 -0600 Subject: [PATCH] Confirm before deleting. --- packages/gallery/src/interface/allImages.js | 6 ++++-- packages/gallery/src/interface/focus.js | 6 ++++-- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/packages/gallery/src/interface/allImages.js b/packages/gallery/src/interface/allImages.js index 51da313..0dffd84 100644 --- a/packages/gallery/src/interface/allImages.js +++ b/packages/gallery/src/interface/allImages.js @@ -98,8 +98,10 @@ export function AllImagesView(vm, params, key, context) { } function deleteSelectedImages() { - selectedIds.forEach(ImageType.delete); - selectedIds.clear(); + if (confirm(`Delete ${selectedIds.size} image(s)?`)) { + selectedIds.forEach(ImageType.delete); + selectedIds.clear(); + } } // function addAlbum() { diff --git a/packages/gallery/src/interface/focus.js b/packages/gallery/src/interface/focus.js index a5e3814..f70b874 100644 --- a/packages/gallery/src/interface/focus.js +++ b/packages/gallery/src/interface/focus.js @@ -50,8 +50,10 @@ export function FocusView(vm, params, key, { appbar }) { } async function clickTrash() { - await ImageType.delete(id()); - navBack(); + if (confirm('Delete this image?')) { + await ImageType.delete(id()); + navBack(); + } } function renderAppBarButtons() {