Merge pull request #543 from sul-dlss/danger_callout

Separate delete button from settings form. Fixes #513

Author: Jessie Keck

app/assets/stylesheets/spotlight/_exhibit_admin.css.scss

@@ -27,4 +27,21 @@
       white-space: nowrap;
     }
   }
+
+  .callout {
+    margin: 20px 0;
+    padding: 20px;
+    border-left: 3px solid #eee;
+    h4 {
+      margin-top: 0;
+      margin-bottom: 5px;
+    }
+  }
+  .callout-danger {
+    background-color: #fdf7f7;
+    border-color: #d9534f;
+    h4 {
+      color: #d9534f;
+    }
+  }
 }

app/views/spotlight/exhibits/edit.html.erb

@@ -1,25 +1,31 @@
 <%= render 'spotlight/shared/curation_sidebar' %>
 <div id="content" class="col-md-9 exhibit-admin">
-<%= administration_page_title %>
-<%= bootstrap_form_for @exhibit, url: spotlight.exhibit_path(@exhibit), style: :horizontal do |f| %>
-  <div class="row col-md-9">
-    <%= f.text_field :title %>
-    <%= f.text_field :subtitle %>
-    <%= f.text_area :description %>
-    <%= f.form_group(:contact_emails, label: { text: nil, class: nil }, help: nil) do %>
-      <%= f.fields_for :contact_emails do |contact| %>
-        <%= render partial: 'contact', locals: {contact: contact} %>
+  <%= administration_page_title %>
+  <%= bootstrap_form_for @exhibit, url: spotlight.exhibit_path(@exhibit), style: :horizontal, html: {class: "row"} do |f| %>
+    <div class="row col-md-9">
+      <%= f.text_field :title %>
+      <%= f.text_field :subtitle %>
+      <%= f.text_area :description %>
+      <%= f.form_group(:contact_emails, label: { text: nil, class: nil }, help: nil) do %>
+        <%= f.fields_for :contact_emails do |contact| %>
+          <%= render partial: 'contact', locals: {contact: contact} %>
+        <% end %>
+        <p class="help-block"><%= t(:'.fields.contact_emails.help_block') %></p>
       <% end %>
-      <p class="help-block"><%= t(:'.fields.contact_emails.help_block') %></p>
-    <% end %>
 
-    <div class="form-actions">
-      <%= delete_link @exhibit, class: 'btn btn-danger' %>
+      <div class="form-actions">
 
-      <div class="primary-actions">
-        <%= f.submit nil, class: 'btn btn-primary' %>
+        <div class="primary-actions">
+          <%= f.submit nil, class: 'btn btn-primary' %>
+        </div>
       </div>
     </div>
+  <% end %>
+  <div class="callout callout-danger row">
+    <h4><%= t(:".delete_exhibit") %></h4>
+    <p class='col-md-9'><%= t(:".delete_exhibit_warning_html", export_link: link_to(t(:'spotlight.exhibits.import.export.download'), spotlight.import_exhibit_path(@exhibit))) %></p>
+    <div class='col-md-3'>
+      <%= delete_link @exhibit, class: 'btn btn-default' %>
+    </div>
   </div>
-<% end %>
 </div>

config/locales/spotlight.en.yml

@@ -215,6 +215,8 @@ en:
         fields:
           contact_emails:
             help_block: Each contact email will receive feedback submissions
+        delete_exhibit: Delete exhibit
+        delete_exhibit_warning_html: This action is irreversable. Be sure to back up the exhibit settings and content using the %{export_link} feature before proceeding.
       new:
         header: Create a new exhibit
       import:

spec/views/spotlight/exhibits/edit.html.erb_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+module Spotlight
+  describe "spotlight/exhibits/edit" do
+    let(:exhibit) { FactoryGirl.create(:exhibit) } 
+    before do
+      assign(:exhibit, exhibit)
+      view.stub(current_exhibit: exhibit)
+    end
+
+    it "renders the edit page form" do
+      render
+
+      expect(rendered).to have_selector "form[action=\"#{spotlight.exhibit_path(exhibit)}\"]"
+      expect(rendered).to have_selector ".callout.callout-danger.row"
+      expect(rendered).to have_content "This action is irreversable"
+      expect(rendered).to have_link "Export", href: spotlight.import_exhibit_path(exhibit)
+
+    end
+  end
+end
+