Improve changeset detail display
authorAaron Lidman <aaronlidman@gmail.com>
Mon, 18 Nov 2013 23:23:48 +0000 (15:23 -0800)
committerAaron Lidman <aaronlidman@gmail.com>
Mon, 18 Nov 2013 23:23:48 +0000 (15:23 -0800)
app/assets/stylesheets/common.css.scss
app/controllers/browse_controller.rb
app/views/user/view.html.erb

index 6468635..60683fc 100644 (file)
@@ -996,7 +996,7 @@ header .search_form {
 
 #sidebar .changesets {
   li {
-    padding: $lineheight;
+    padding: 10px 20px;
     border-bottom: 1px solid #ddd;
     cursor: pointer;
 
index c8375ee..7380ac5 100644 (file)
@@ -58,8 +58,8 @@ class BrowseController < ApplicationController
     @type = "changeset"
     @changeset = Changeset.find(params[:id])
     @node_pages, @nodes = paginate(:old_nodes, :conditions => {:changeset_id => @changeset.id}, :per_page => 10, :parameter => 'node_page')
-    @way_pages, @ways = paginate(:old_ways, :conditions => {:changeset_id => @changeset.id}, :per_page => 10, :parameter => 'way_page')
-    @relation_pages, @relations = paginate(:old_relations, :conditions => {:changeset_id => @changeset.id}, :per_page => 10, :parameter => 'relation_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
     render :action => "not_found", :status => :not_found
   end
index 104b1ec..f39bafe 100644 (file)
@@ -4,10 +4,10 @@
     <div class='userinformation-inner'>
       <h1><%= @this_user.display_name %><%= role_icons(@this_user) %></h1>
       <% if @user and @this_user.id == @user.id %>
-        <!-- Displaying user's own profile page -->
+        <!-- Displaying user's own profile page to themself -->
         <ul class='secondary-actions clearfix'>
           <li>
-            <%= link_to t('user.view.my edits'), :controller => 'changeset', :action => 'list', :display_name => @user.display_name, :anchor => "map=1/0/0" %>
+            <%= link_to t('user.view.my edits'), :controller => 'changeset', :action => 'list', :display_name => @user.display_name %>
             <span class='count-number'><%= number_with_delimiter(@user.changesets.size) %></span>
           </li>
           <li>
         </ul>
 
         <% else %>
-
+        <!-- Displaying user profile page to the public -->
         <ul class='secondary-actions clearfix'>
 
           <li>
-            <%= link_to t('user.view.edits'), :controller => 'changeset', :action => 'list', :display_name => @this_user.display_name, :anchor => "map=1/0/0" %>
+            <%= link_to t('user.view.edits'), :controller => 'changeset', :action => 'list', :display_name => @this_user.display_name %>
             <span class='count-number'><%= number_with_delimiter(@this_user.changesets.size) %></span>
           </li>
           <li>