From 5616bc2053c8773147acfbf68a48c867656d49e6 Mon Sep 17 00:00:00 2001
From: Tom Rees <zephod@gmail.com>
Date: Thu, 26 Jan 2012 19:15:34 +0000
Subject: [PATCH] [#1425][m]: Redesigned front-end for dataset deletion. Closes
 #1425.

---
 ckan/public/css/style.css                    |  3 +++
 ckan/public/scripts/application.js           | 16 +++++++++++++--
 ckan/templates/package/edit.html             |  1 +
 ckan/templates/package/new_package_form.html | 21 ++++++++++++--------
 4 files changed, 31 insertions(+), 10 deletions(-)

diff --git a/ckan/public/css/style.css b/ckan/public/css/style.css
index 5b6cf79299c..34a2ec04046 100644
--- a/ckan/public/css/style.css
+++ b/ckan/public/css/style.css
@@ -963,6 +963,9 @@ ul.dataset-edit-nav li a:hover {
 .dataset-edit-form .resource-add .fileinfo {
   margin: 7px 0;
 }
+.dataset-edit-form button.dataset-delete {
+  vertical-align: top;
+}
 
 
 /* ================================ */
diff --git a/ckan/public/scripts/application.js b/ckan/public/scripts/application.js
index 2c417429988..d7d3a0825b5 100644
--- a/ckan/public/scripts/application.js
+++ b/ckan/public/scripts/application.js
@@ -80,6 +80,18 @@
         el: $el
       });
       view.render();
+
+      // Set up dataset delete button
+      var select = $('select.dataset-delete');
+      select.attr('disabled','disabled');
+      select.css({opacity: 0.3});
+      $('button.dataset-delete').click(function(e) {
+        select.removeAttr('disabled');
+        select.fadeTo('fast',1.0);
+        $(e.target).css({opacity:0});
+        $(e.target).attr('disabled','disabled');
+        return false;
+      });
     }
     var isGroupEdit = $('body.group.edit').length > 0;
     if (isGroupEdit) {
@@ -497,7 +509,7 @@ CKAN.View.DatasetEditForm = Backbone.View.extend({
           parentDiv.append(messageDiv);
           $('#unsaved-warning').append(parentDiv);
           console.log($('#unsaved-warning'));
-          messageDiv.show(1200);
+          messageDiv.show(200);
 
           boundToUnload = true;
           window.onbeforeunload = function () { 
@@ -507,7 +519,7 @@ CKAN.View.DatasetEditForm = Backbone.View.extend({
       }
     }();
 
-    $form.find('input').live('change', function(e) {
+    $form.find('input,select').live('change', function(e) {
       $target = $(e.target);
       // Entering text in the 'add' box does not represent a change
       if ($target.closest('.resource-add').length==0) {
diff --git a/ckan/templates/package/edit.html b/ckan/templates/package/edit.html
index 1028975da88..6bd9a39735c 100644
--- a/ckan/templates/package/edit.html
+++ b/ckan/templates/package/edit.html
@@ -23,6 +23,7 @@
         <li><a href="#section-resources">Resources</a></li>
         <li><a href="#section-groups">Groups &amp; Tags</a></li>
         <li><a href="#section-extras">Extras</a></li>
+        <li py:if="c.is_sysadmin or c.auth_for_change_state"><a href="#section-delete">Delete</a></li>
       </ul>
     </li>
   </py:match>
diff --git a/ckan/templates/package/new_package_form.html b/ckan/templates/package/new_package_form.html
index bbf2f5f2b7c..788e590162c 100644
--- a/ckan/templates/package/new_package_form.html
+++ b/ckan/templates/package/new_package_form.html
@@ -165,14 +165,6 @@ <h3>Tags</h3>
     <dd class="instructions basic">A number representing the version (if applicable)</dd>
     <dd class="hints">e.g. 1.2.0</dd>
 
-    <dt py:if="c.is_sysadmin or c.auth_for_change_state"><label class="field_opt" for="state">State</label></dt>
-    <dd py:if="c.is_sysadmin or c.auth_for_change_state">
-    <select id="state" name="state" >
-      <option py:attrs="{'selected': 'selected' if data.get('state') == 'active' else None}" value="active">active</option>
-      <option py:attrs="{'selected': 'selected' if data.get('state') == 'deleted' else None}" value="deleted">deleted</option>
-    </select>
-    </dd>
-
   </dl>
 </fieldset>
 
@@ -199,6 +191,19 @@ <h3>Tags</h3>
     </py:with>
   </dl>
 </fieldset>
+<fieldset id='delete' py:if="c.is_sysadmin or c.auth_for_change_state">
+  <dl>
+    <dt>Delete</dt>
+    <dd>
+      <p>Do you really want to change the state of this dataset? &nbsp;&nbsp;<button class="dataset-delete pretty-button">Yes!</button></p>
+      This dataset is&nbsp;&nbsp; 
+      <select class="dataset-delete" id="state" name="state" style="display:inline;">
+        <option py:attrs="{'selected': 'selected' if data.get('state') == 'active' else None}" value="active">active</option>
+        <option py:attrs="{'selected': 'selected' if data.get('state') == 'deleted' else None}" value="deleted">deleted</option>
+      </select>
+    </dd>
+  </dl>
+</fieldset>
 
 <div id="unsaved-warning"></div>