Fix lists in rich text to display properly

[rails.git] / app / views / browse / _navigation.html.erb

diff --git a/app/views/browse/_navigation.html.erb b/app/views/browse/_navigation.html.erb
index b125ac75fceb92a6e2414ed832cb155c66147a83..0856044c8087599ef6ae1b902832e5d463bca04c 100644 (file)
--- a/app/views/browse/_navigation.html.erb
+++ b/app/views/browse/_navigation.html.erb
@@ -1,36 +1,43 @@
-<div style="float:right; text-align:center; width: 250px;">
-  <% if @next_by_user or @prev_by_user %>
-    <% if @prev_by_user %>
-       <%= link_to t('browse.changeset_navigation.user.prev', :id => @prev_by_user.id.to_s),
-           { :id => @prev_by_user.id },
-           { :title => t('browse.changeset_navigation.user.prev_tooltip', :user => @prev_by_user.user.display_name) } %>
-    |
+<% type = (@next || @prev).class.name.downcase %>
+  <ul class='secondary-actions clearfix pager'>
+    <% if @prev %>
+    <li>
+      <%= link_to t('browse.navigation.paging.all.prev', :id => @prev.id.to_s),
+                  { :id => @prev.id },
+                  { :title => t("browse.navigation.all.prev_#{type}_tooltip") } %>
+    </li>
     <% end %>
-    <%=
-    user = (@prev_by_user || @next_by_user).user.display_name
-    link_to(t('browse.changeset_navigation.user.name', :user => h(user)),
-            { :controller => "changeset", :action => "list", :display_name => user },
-            { :title => t('browse.changeset_navigation.user.name_tooltip', :user => h(user)) })
-    %>
-    <% if @next_by_user %>
-    |
-      <%= link_to t('browse.changeset_navigation.user.next', :id => @next_by_user.id.to_s),
-         { :id => @next_by_user.id },
-         { :title => t('browse.changeset_navigation.user.next_tooltip', :user => @next_by_user.user.display_name) } %>
+    <% if @next %>
+      <li>
+        <%= link_to t('browse.navigation.paging.all.next', :id => @next.id.to_s),
+                    { :id => @next.id },
+                    { :title => t("browse.navigation.all.next_#{type}_tooltip") } %>
+      </li>
     <% end %>
-    <br/>
-  <% end %>
-  <% if @prev %>
-    <%= link_to t('browse.changeset_navigation.all.prev', :id => @prev.id.to_s),
-                { :id => @prev.id },
-                { :title => t('browse.changeset_navigation.all.prev_tooltip') } %>
-  <% end %>
-  <% if @prev and @next %>
-    | 
-  <% end %>
-  <% if @next %>
-    <%= link_to t('browse.changeset_navigation.all.next', :id => @next.id.to_s),
-                { :id => @next.id },
-                { :title => t('browse.changeset_navigation.all.next_tooltip') } %>
+  </ul>
+  <% if @next_by_user or @prev_by_user %>
+    <ul class='secondary-actions pager clearfix'>
+      <% if @prev_by_user %>
+        <li>
+            <%= link_to t('browse.navigation.paging.user.prev', :id => @prev_by_user.id.to_s),
+                        { :id => @prev_by_user.id },
+                        { :title => t("browse.navigation.user.prev_#{type}_tooltip", :user => @prev_by_user.user.display_name) } %>
+        </li>
+      <% end %>
+        <li>
+          <%=
+          user = (@prev_by_user || @next_by_user).user.display_name
+          link_to h(user),
+                  { :controller => "changeset", :action => "list", :display_name => user },
+                  { :title => t("browse.navigation.user.name_#{type}_tooltip", :user => h(user)) }
+          %>
+        </li>
+      <% if @next_by_user %>
+        <li>
+          <%= link_to t('browse.navigation.paging.user.next', :id => @next_by_user.id.to_s),
+                      { :id => @next_by_user.id },
+                      { :title => t("browse.navigation.user.next_#{type}_tooltip", :user => @next_by_user.user.display_name) } %>
+        </li>
+      <% end %>
+    </ul>
   <% end %>
-</div>