From ea189735e8f4d0affcd81a3a8b1c47a66b30d118 Mon Sep 17 00:00:00 2001
From: Tom Rees <zephod@gmail.com>
Date: Thu, 1 Mar 2012 18:05:15 +0000
Subject: [PATCH] [#1506][resources][m]: Refactoring look-and-feel.

---
 ckan/public/css/style.css                    | 39 ++++++++++++--------
 ckan/templates/package/new_package_form.html |  3 +-
 ckan/templates/package/read_core.html        |  2 +-
 3 files changed, 26 insertions(+), 18 deletions(-)

diff --git a/ckan/public/css/style.css b/ckan/public/css/style.css
index 7f54fa83058..915a6946bc1 100644
--- a/ckan/public/css/style.css
+++ b/ckan/public/css/style.css
@@ -1536,35 +1536,37 @@ body.editresources fieldset#resources > h3 {
   margin: 0;
 }
 .resource-list li {
-  margin-bottom: 4px;
   background: #fff;
-  border-radius: 4px;
+  border: 0;
   border: 1px solid #eee;
+  border-top-color: transparent;
+  border-left-color: transparent;
+  border-right: 0;
   position: relative;
-  margin-right: 20px;
+  margin: 0 20px 0 0;
   z-index: 1;
   white-space: nowrap;
   overflow: hidden;
+  font-weight: bold;
+}
+.resource-list li:last-child {
+  border-bottom-color: transparent;
 }
 .resource-list li a {
   display: block;
   padding: 5px 10px;
+  color: #333;
+}
+.resource-list li:hover a{
+  color: #B22;
 }
 .resource-list li:hover {
-  border-color: #ccc;
+  background: #f7f7f7;
 }
 .resource-list li.active {
   border-color: #888;
-  border-right: 0;
   background: #f9f9f9;
   margin-right: 0;
-  border-top-right-radius: 0;
-  border-bottom-right-radius: 0;
-}
-hr.resource-list-divide {
-  margin: 10px 0;
-  width: 380px;
-  background: #eee;
 }
 
 
@@ -1590,18 +1592,25 @@ body.editresources .error-explanation {
 
 /* While dragging.... */
 .resource-list-edit li.ui-sortable-helper {
-  box-shadow: 2px 2px 2px rgba(0,0,0,0.1);
+  box-shadow: 2px 2px 8px rgba(0,0,0,0.1);
 }
 .resource-list-edit li.ui-sortable-helper.active {
   margin-right: 20px;
-  border-top-right-radius: 4px;
-  border-bottom-right-radius: 4px;
   border-right: 1px solid #888;
 }
 .resource-list-add,
 .resource-list-edit {
   width: 400px;
 }
+.resource-list-add {
+  margin-top: 20px;
+}
+.resource-list-add li {
+  border-top: 1px solid #eee;
+}
+.resource-list-add li.active {
+  border-bottom-color: #888;
+}
 .resource-list-add li a {
   padding-left: 43px;
 }
diff --git a/ckan/templates/package/new_package_form.html b/ckan/templates/package/new_package_form.html
index e514db052bf..743b07e2924 100644
--- a/ckan/templates/package/new_package_form.html
+++ b/ckan/templates/package/new_package_form.html
@@ -136,9 +136,8 @@ <h3>Add resources:</h3>
   </div>
   <ul class="resource-list resource-list-edit drag-drop-list">
   </ul>
-  <hr class="resource-list-divide"/>
   <ul class="resource-list resource-list-add">
-    <li class="ui-state-default"><a href="#" class="js-resource-add">${h.icon('page_white_add')}New resource...</a></li>
+    <li><a href="#" class="js-resource-add">${h.icon('page_white_add')}New resource...</a></li>
   </ul>
   <div style="display: none;" class="resource-panel">
     <button class="pretty-button danger resource-panel-close">x</button>
diff --git a/ckan/templates/package/read_core.html b/ckan/templates/package/read_core.html
index fdbe060151f..381b14986b8 100644
--- a/ckan/templates/package/read_core.html
+++ b/ckan/templates/package/read_core.html
@@ -29,7 +29,7 @@ <h3>Resources
         </py:if></h3>
         <ul class="resource-list">
           <py:for each="res in c.pkg_dict.get('resources', [])">
-          <li class="ui-state-default">
+          <li>
             <a href="${h.url_for(controller='package', action='resource_read', id=c.pkg_dict['name'], resource_id=res['id'])}">
               ${h.resource_icon(res)}
               ${h.resource_display_name(res)}