Use select_tag for issue filter fields instead of abusing select

[rails.git] / app / views / diary_entry / list.html.erb

diff --git a/app/views/diary_entry/list.html.erb b/app/views/diary_entry/list.html.erb
index a78b41a4f5039f2f7a1e1ddd7086f7595c1334f4..d6af432fb01d485adfa7cdc7ec6585fbd8fdb9b0 100644 (file)
--- a/app/views/diary_entry/list.html.erb
+++ b/app/views/diary_entry/list.html.erb
@@ -1,25 +1,32 @@
-<% if @this_user %>
-  <%= user_image @this_user %>
-<% end %>
-
-<h2><%= h(@title) %></h2>
-
-<% if @this_user %>
-  <% if_user(@this_user) do %>
-    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %>
-  <% end %>
-<% else %>
-  <% if_logged_in do %>
-    <%= link_to image_tag("new.png", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %>
-  <% end %>
+<% content_for :heading do %>
+  <div <% if @this_user %> id="userinformation"<% end %> >
+    <% if @this_user %>
+      <%= user_image @this_user %>
+    <% end %>
+    <h1><%= h(@title) %></h1>
+
+    <ul class='secondary-actions clearfix'>
+      <% unless params[:friends] or params[:nearby] -%>
+        <li><%= rss_link_to :action => 'rss', :language => params[:language] %></li>
+      <% end -%>
+
+      <% if @this_user %>
+        <%= if_user(@this_user) do %>
+          <li><%= link_to image_tag("new.png", :class => "small_icon", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %></li>
+        <% end %>
+      <% else %>
+        <%= if_logged_in do %>
+          <li><%= link_to image_tag("new.png", :class => "small_icon", :border=>0) + t('diary_entry.list.new'), {:controller => 'diary_entry', :action => 'new'}, {:title => t('diary_entry.list.new_title')} %></li>
+        <% end %>
+      <% end %>
+    </ul>
+  </div>
 <% end %>
 
 <% if @entries.empty? %>
-  <p><%= t 'diary_entry.list.no_entries' %></p>
+  <h4><%= t 'diary_entry.list.no_entries' %></h4>
 <% else %>
-  <p><%= t 'diary_entry.list.recent_entries' %></p>
-
-  <hr />
+  <h4><%= t 'diary_entry.list.recent_entries' %></h4>
 
   <% if @this_user %>
     <%= render :partial => 'diary_entry', :collection => @entries %>
@@ -27,15 +34,25 @@
     <%= render :partial => 'diary_list_entry', :collection => @entries %>
   <% end %>
 
-  <%= link_to t('diary_entry.list.older_entries'), { :page => @entry_pages.current.next, :language => params[:language] } if @entry_pages.current.next %>
-  <% if @entry_pages.current.next and @entry_pages.current.previous %>|<% end %>
-  <%= link_to t('diary_entry.list.newer_entries'), { :page => @entry_pages.current.previous, :language => params[:language] } if @entry_pages.current.previous %>
-
-  <br />
+  <div class="pagination">
+    <% if @entries.size < @page_size -%>
+      <%= t('diary_entry.list.older_entries') %>
+    <% else -%>
+      <%= link_to t('diary_entry.list.older_entries'), @params.merge(:page => @page + 1 ) %>
+    <% end -%>
+
+    |
+
+    <% if @page > 1 -%>
+      <%= link_to t('diary_entry.list.newer_entries'), @params.merge(:page => @page - 1) %>
+    <% else -%>
+      <%= t('diary_entry.list.newer_entries') %>
+    <% end -%>
+  </div>
 <% end %>
 
-<%= rss_link_to :action => 'rss', :language => params[:language] %>
-
-<% content_for :head do %>
-<%= auto_discovery_link_tag :atom, :action => 'rss', :language => params[:language] %>
-<% end %>
+<% unless params[:friends] or params[:nearby] -%>
+  <% content_for :auto_discovery_link_tag do -%>
+  <%= auto_discovery_link_tag :rss, :action => 'rss', :language => params[:language] %>
+  <% end -%>
+<% end -%>