deploy key creation restyled

app/assets/stylesheets/style.scss

@@ -220,13 +220,9 @@ input.button{margin-bottom: 1.5em}
 
 .ui-box .data{padding: .5em 1em}
 
-.ui-box .buttons{background-color: #f7f8f9; padding: 1em;
-  -webkit-border-bottom-right-radius: 5px;
-  -webkit-border-bottom-left-radius: 5px;
-  -moz-border-radius-bottomright: 5px;
-  -moz-border-radius-bottomleft: 5px;
-  border-bottom-right-radius: 5px;
-  border-bottom-left-radius: 5px;  
+.ui-box .buttons{
+  padding: 1em;
+  border-top:1px solid $lite_border_color;
 }
 
 .ui-box .buttons .button{padding: 8px 9px; font-size: 11px}
@@ -713,3 +709,17 @@ body, button, input, select, textarea {
   }
 }
 
+
+.errors_holder {
+  background:#FBB;
+  border:1px solid $lite_border_color;
+  @include round-borders-all(4px);
+  padding:5px;
+  list-style:none;
+  font-weight: bold;
+  text-shadow: 0 -1px 0 rgba(255, 255, 255, 0.25);
+
+  li { 
+    padding:10px;
+  }
+}

app/controllers/deploy_keys_controller.rb

 class DeployKeysController < ApplicationController
-  respond_to :js, :html
+  respond_to :html
   layout "project"
   before_filter :project
 
@@ -29,7 +29,7 @@ class DeployKeysController < ApplicationController
     @key = @project.deploy_keys.new(params[:key])
     @key.save
 
-    respond_with(@key)
+    redirect_to project_deploy_keys_path(@project)
   end
 
   def destroy

app/views/deploy_keys/_form.html.haml

 %div
-  = form_for [@project, @key], :url => project_deploy_keys_path, :remote => true do |f|
+  = form_for [@project, @key], :url => project_deploy_keys_path do |f|
     -if @key.errors.any?
-      %ul
+      %ul.errors_holder
         - @key.errors.full_messages.each do |msg|
           %li= msg
     
-    .form-row
-      = f.label :title
-      = f.text_field :title, :style => "width:300px"
-    .form-row
-      = f.label :key
-      = f.text_area :key, :style => "width:300px; height:130px"
-    .form-row
+    %table.no-borders
+      %tr
+        %td= f.label :title
+        %td= f.text_field :title, :style => "width:300px"
+      %tr
+        %td= f.label :key
+        %td= f.text_area :key, :style => "width:300px; height:130px"
+    %br
+    .merge-tabs
       = f.submit 'Save', :class => "grey-button"
 

app/views/deploy_keys/create.js.haml

-- if @key.valid?
-  :plain
-    $("#new_key_dialog").dialog("close");
-    $("#keys-table .data").append("#{escape_javascript(render(:partial => 'show', :locals => {:key => @key} ))}");
-    $("#no_ssh_key_defined").hide();
-- else
-  :plain
-    $("#new_key_dialog").empty();
-    $("#new_key_dialog").append("#{escape_javascript(render('form'))}");

app/views/deploy_keys/edit.html.haml

-%h1 Editing key
-
-= render 'form'
-
-= link_to 'Show', @key
-\|
-= link_to 'Back', project_deploy_keys_path

app/views/deploy_keys/new.html.haml

-%h1 New key
+= render "repositories/head"
 
-= render 'form'
+%h2 New Deploy key
 
-= link_to 'Back', project_deploy_keys_path
+= render 'form'

app/views/deploy_keys/new.js.haml

-:plain
-  var new_key_dialog = $("<div id='new_key_dialog'></div>");
-  new_key_dialog.html("#{escape_javascript(render('form'))}");
-  $(new_key_dialog).dialog({
-    width: 350,
-    resizable: false,
-    draggable: false,
-    title: "Add new public key",
-    close: function(event, ui) { $("#new_key_dialog").remove();},
-    modal: true
-  });

app/views/issues/_form.html.haml

@@ -20,18 +20,13 @@
                 %i
 
       %h2= @issue.new_record? ? "New Issue" : "Edit Issue ##{@issue.id}"
-
       %hr
-    %table.no-borders
     -if @issue.errors.any?
-        %tr 
-          %td{:colspan => 2}
-            #error_explanation
+      %ul.errors_holder
         - @issue.errors.full_messages.each do |msg|
-                %span= msg
-                %br
-
+          %li= msg
 
+    %table.no-borders
       %tr
         %td= f.label :assignee_id
         %td= f.select(:assignee_id, @project.users.all.collect {|p| [ p.name, p.id ] }, { :include_blank => "Select user" })

app/views/repositories/_head.html.haml

@@ -23,6 +23,6 @@
 
   - if current_page?(project_deploy_keys_path(@project))
     - if can? current_user, :admin_project, @project
-      = link_to new_project_deploy_key_path(@project), :class => "add_new", :title => "New Deploy Key", :remote => true do 
+      = link_to new_project_deploy_key_path(@project), :class => "add_new", :title => "New Deploy Key" do 
         = image_tag "add_new.png", :width => 14
 

spec/requests/projects_deploy_keys_spec.rb

@@ -36,7 +36,7 @@ describe "Projects", "DeployKeys" do
     end
 
     it "should open new key popup" do
-      page.should have_content("Add new public key")
+      page.should have_content("New Deploy key")
     end
 
     describe "fill in" do
@@ -50,7 +50,6 @@ describe "Projects", "DeployKeys" do
       it "should add new key to table" do
         click_button "Save"
 
-        page.should_not have_content("Add new public key")
         page.should have_content "laptop"
       end
     end