ActionItem21: another checkpoint

git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@971 3f533792-8f58-4932-b0fe-aaf55b0a4547

ActionItem21: another checkpoint
git-svn-id: https://svn.colivre.coop.br/svn/noosfero/trunk@971 3f533792-8f58-4932-b0fe-aaf55b0a4547
AntonioTerceiro
1 parent 37c662dc
Showing 5 changed files with 36 additions and 29 deletions Show diff stats
app/helpers/application_helper.rb
app/views/cms/view.rhtml
public/designs/icons/default/abiword_48.png
public/stylesheets/button.css
public/stylesheets/common.css
@@ -346,7 +346,11 @@ module ApplicationHelper
   end
   def icon(icon_name)
-    design_display_icon(icon_name, :style => 'width: 24px; height: 24px; display: inline;')
+    design_display_icon(icon_name)
+  end
+
+  def button_bar(options = {}, &block)
+    concat(content_tag('div', capture(&block) + tag('br', :style => 'clear: left;'), options), block.binding)
   end
 end
@@ -9,45 +9,46 @@
     <%= icon('cms') %>
     <%= _('Content management') %>
   </h2>
-  <div style='margin-bottom: 1em;'>
+
+  <% button_bar(:style => 'margin-bottom: 1em;') do %>
     <%= button('new', _('New article'), :action => 'new') %>
-  </div>
+  <% end %>
+
 <% end %>
 <%# subitem %>
+
 <%= toggle_panel(_('Hide subitems'), _('Show subitems'), 'article-subitems') unless @subitems.empty? %>
+  
 <div id='article-subitems'>
-    <div class='file-manager-title'><%= @article ? _('Subitems') : _('Articles') %></div>
-    <div class='file-manager-small'>
-      <% unless @subitems.empty? %>
-        <ul>
-          <% @subitems.each do |item| %>
-            <li>
-            <%= file_manager_button(item.name, "icons-mime/#{item.icon_name}", :action => 'view', :id => item.id) %>
-            </li>
-          <% end %>
-        </ul>
-      <% end %>
+  <div class='file-manager-title'><%= @article ? _('Subitems') : _('Articles') %></div>
+  <div class='file-manager-small'>
+    <% unless @subitems.empty? %>
+      <ul>
+        <% @subitems.each do |item| %>
+          <li>
+          <%= file_manager_button(item.name, "icons-mime/#{item.icon_name}", :action => 'view', :id => item.id) %>
+          </li>
+        <% end %>
+      </ul>
+    <% end %>
-      <% if @article %>
-        <div class='file-manager-controls'>
-          <%= button('new',  _('New subitem'), :action => 'new', :parent_id => @article.id) %>
-        </div>
+    <% if @article %>
+      <% button_bar(:class => 'file-manager-controls') do %>
+        <%= button('new',  _('New subitem'), :action => 'new', :parent_id => @article.id) %>
       <% end %>
-
-    </div>
+    <% end %>
+  </div>
 </div> <!-- article-children -->
 <%# display the article content %>
-<div id='article-contents'>
+<div id='article-contents' style='clear: left;'>
   <% if @article %>
     <h2><%= @article.name %></h2>
-    <div id='article-controls'>
-
+    <% button_bar(:id => 'article-controls') do %>
       <%= link_to_edit_article(@article) %>
-
       <%= button('home', _('Use as homepage'), { :action => 'set_home_page', :id => @article }, { :method => :post  }) %>
-    </div>
+    <% end %>
     <%= @article.to_html %>
   <% end %>
 </div>
@@ -6,6 +6,10 @@
   background-position: 0px;
   padding-left: 25px;
+
+  float: left;
+  display: block;
+  height: 26px;
 }
 a.button , a.button:visited {
@@ -17,4 +21,3 @@ a.button:hover {
   background-color: #ace;
 }
-
@@ -198,7 +198,6 @@ div.file-manager-button a:hover {
 #article-subitems-hide.hide-button {
   float: right;
-  margin-top: 2px;
   margin-right: 4px;
   color: white;
 }
@@ -219,6 +218,6 @@ div.file-manager-button a:hover {
 #article-controls {
   background: white;
   border: 1px solid gray;
-  padding-top: 8px;
-  padding-bottom: 8px;
+  padding-top: 4px;
+  padding-bottom: 4px;
 }