General improvements to the layout of the hover item

baphled · Jun 5, 2010 · 0481e36 · 0481e36
1 parent 6bbe8af
commit 0481e36
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 7 deletions.
diff --git a/app/stylesheets/partials/_hover_dialog.sass b/app/stylesheets/partials/_hover_dialog.sass
@@ -1,8 +1,9 @@
 .hover
   position: relative
-  width: 418px
-  padding: 5px
   top: 10px
+  margin: 13px 13px auto auto !important
+  h3
+    text-align: center
 .title-fixed
   position: fixed
   top: 0

diff --git a/app/stylesheets/screen.sass b/app/stylesheets/screen.sass
@@ -124,8 +124,6 @@ body.salad
       +no-bullets
       :height auto
       padding-top: 49px
-      .side_hover
-        padding-top: 10px
       li
         :padding 5px 0px
     #panel

diff --git a/app/views/common/_sidebar_list_content.html.erb b/app/views/common/_sidebar_list_content.html.erb
@@ -1,8 +1,9 @@
 <% if !models.blank? %>
-  <ul id="<%= models.first.class.to_s.singularize.downcase %>_<%=assoc%>">
+  <ul id="<%= models.first.class.to_s.singularize.downcase %>_<%=assoc%>" class="ui-widget ui-widget-content ui-corner-all hover">
   <% models.each_with_index do |model, index| %>
       <li id="<%= models.first.class.to_s.singularize.downcase %>_<%=model.id%>_<%=assoc%>" class="<%= models.first.class.to_s.singularize.downcase %>_list_item side_hover" style="display: none;">
         <h3><%=h display_item_title(model) %></h3>
+        <hr/>
       <% if !model.send(assoc.to_sym).blank? %>
         <ul>
           <% model.send(assoc.to_sym).each_with_index do |item, item_index|%>

diff --git a/public/javascripts/item_hover.js b/public/javascripts/item_hover.js
@@ -3,7 +3,7 @@ $(document).ready(function() {
   $.fn.autoScroller = function(speed,height) {
     var width = $(this).parent().width();
     var $innerUnorderedList = $(this).find('ul');
-    $innerUnorderedList.css('height', height)
+    $innerUnorderedList.css('max-height', height)
                        .css('overflow-y', 'hidden');
 
     $(this).parent().css('width', width);
@@ -27,7 +27,7 @@ $(document).ready(function() {
     var sidebarPostfix = $('div#sidebar > ul').attr('id').split('_');
     var $sidebar = $('div#sidebar > ul');
     var hoverItem = 'li#' + $(this).attr('id') + '_' + sidebarPostfix[1];
-    var height = $('div#lists').outerHeight();
+    var height = $('div#lists').outerHeight() - $('div#footer').outerHeight();
 
     if (!$(hoverItem).is(':visible')) {
       $('div#sidebar').find('li.side_hover').hide();