Reorder changeset detail lists

Nodes, ways, relations -> Ways, relations, nodes

diff --git a/app/controllers/browse_controller.rb b/app/controllers/browse_controller.rb
index 7380ac5bb0b83f8fe288821153f37cc44c078f7f..29cd6707f5f847346144e96b31d0b7a22d6ee343 100644 (file)
--- a/app/controllers/browse_controller.rb
+++ b/app/controllers/browse_controller.rb
@@ -57,7 +57,7 @@ class BrowseController < ApplicationController
   def changeset
     @type = "changeset"
     @changeset = Changeset.find(params[:id])
   def changeset
     @type = "changeset"
     @changeset = Changeset.find(params[:id])

-    @node_pages, @nodes = paginate(:old_nodes, :conditions => {:changeset_id => @changeset.id}, :per_page => 10, :parameter => 'node_page')
+    @node_pages, @nodes = paginate(:old_nodes, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'node_page')

     @way_pages, @ways = paginate(:old_ways, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'way_page')
     @relation_pages, @relations = paginate(:old_relations, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'relation_page')
   rescue ActiveRecord::RecordNotFound
     @way_pages, @ways = paginate(:old_ways, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'way_page')
     @relation_pages, @relations = paginate(:old_relations, :conditions => {:changeset_id => @changeset.id}, :per_page => 20, :parameter => 'relation_page')
   rescue ActiveRecord::RecordNotFound

diff --git a/app/views/browse/changeset.html.erb b/app/views/browse/changeset.html.erb
index ee8948d9bffa01c246060ce5ca9878e380c63095..aa333cfff72876e34c16db4767b9630d2c057dce 100644 (file)
--- a/app/views/browse/changeset.html.erb
+++ b/app/views/browse/changeset.html.erb
@@ -11,18 +11,6 @@
 
   <%= render :partial => "tag_details", :object => @changeset.tags.except('comment') %>
 
 
   <%= render :partial => "tag_details", :object => @changeset.tags.except('comment') %>
 

-  <% unless @nodes.empty? %>
-    <h4>
-      <%= type_and_paginated_count('node', @node_pages) %>
-      <%= render :partial => 'paging_nav', :locals => { :pages => @node_pages, :page_param => "node_page"} %>
-    </h4>
-    <ul>
-      <% @nodes.each do |node| %>
-        <li><%= link_to h(printable_name(node, true)), { :action => "node", :id => node.node_id.to_s }, :class => link_class('node', node), :title => link_title(node) %></li>
-      <% end %>
-    </ul>
-  <% end %>
-

   <% unless @ways.empty? %>
     <h4>
       <%= type_and_paginated_count('way', @way_pages) %>
   <% unless @ways.empty? %>
     <h4>
       <%= type_and_paginated_count('way', @way_pages) %>


@@ -46,6 +34,18 @@
       <% end %>
     </ul>
   <% end %>
       <% end %>
     </ul>
   <% end %>

+
+  <% unless @nodes.empty? %>
+    <h4>
+      <%= type_and_paginated_count('node', @node_pages) %>
+      <%= render :partial => 'paging_nav', :locals => { :pages => @node_pages, :page_param => "node_page"} %>
+    </h4>
+    <ul>
+      <% @nodes.each do |node| %>
+        <li><%= link_to h(printable_name(node, true)), { :action => "node", :id => node.node_id.to_s }, :class => link_class('node', node), :title => link_title(node) %></li>
+      <% end %>
+    </ul>
+  <% end %>

 </div>
 
 <div class='secondary-actions'>
 </div>
 
 <div class='secondary-actions'>