From 8bf26ea1a58d6913fa6820f2305d009c39fc3444 Mon Sep 17 00:00:00 2001 From: Christoph Haas Date: Tue, 12 Jul 2016 20:31:24 +0200 Subject: [PATCH] Added Javascript confirmation upon deleting images --- app/views/moderate/index.slim | 4 ++-- app/views/packages/upload.slim | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/views/moderate/index.slim b/app/views/moderate/index.slim index 9ce0f98..1ae0824 100644 --- a/app/views/moderate/index.slim +++ b/app/views/moderate/index.slim @@ -26,13 +26,13 @@ - if screenshot.approved==false div / TODO: Add a reason for the rejection to tell the user - a.button.alert.round href=delete_screenshot_path(screenshot) Reject + a.button.alert.round href=delete_screenshot_path(screenshot) onclick="return confirm('Really delete the screenshot?');" Reject a.button.success.round href=approve_screenshot_path(screenshot) Approve / Existing screenshots that was requested for removal - if screenshot.markedfordelete==true .button-group - a.button.alert.round href=delete_screenshot_path(screenshot) Remove as requested + a.button.alert.round href=delete_screenshot_path(screenshot) onclick="return confirm('Really delete the screenshot?');" Remove as requested a.button.success.round href=approve_screenshot_path(screenshot) Keep the screenshot diff --git a/app/views/packages/upload.slim b/app/views/packages/upload.slim index 28cfa99..cdf5372 100644 --- a/app/views/packages/upload.slim +++ b/app/views/packages/upload.slim @@ -24,7 +24,7 @@ - @package.screenshots.each do |screenshot| .row img src=screenshot.image.url(:large, timestamp: false, alt: screenshot.caption) - = link_to "Delete screenshot", delete_screenshot_path(@package.name, screenshot.id) + = link_to "Delete screenshot", delete_screenshot_path(@package.name, screenshot.id), onclick: "return confirm('Really delete the screenshot?');" // - @package.screenshots.each do |screenshot| // .listview