From: Tom Hughes <tom@compton.nu>
Date: Wed, 8 Jul 2020 17:57:32 +0000 (+0100)
Subject: Merge remote-tracking branch 'upstream/pull/2697'
X-Git-Tag: live~3059
X-Git-Url: https://git.openstreetmap.org./rails.git/commitdiff_plain/3a43f9fe0421afdf74d2f570ee8987a4022519cf?hp=-c

Merge remote-tracking branch 'upstream/pull/2697'
---

3a43f9fe0421afdf74d2f570ee8987a4022519cf
diff --combined app/views/users/show.html.erb
index 35ee7776b,5b8dcfcc9..4e48ee983
--- a/app/views/users/show.html.erb
+++ b/app/views/users/show.html.erb
@@@ -11,7 -11,7 +11,7 @@@
              <span class='count-number'><%= number_with_delimiter(current_user.changesets.size) %></span>
            </li>
            <li>
 -            <%= link_to t(".my notes"), :controller => "notes", :action => "mine" %>
 +            <%= link_to t(".my notes"), user_notes_path(@user) %>
            </li>
            <li>
              <%= link_to t(".my traces"), :controller => "traces", :action => "mine" %>
@@@ -53,7 -53,7 +53,7 @@@
              <span class='count-number'><%= number_with_delimiter(@user.changesets.size) %></span>
            </li>
            <li>
 -            <%= link_to t(".notes"), :controller => "notes", :action => "mine" %>
 +            <%= link_to t(".notes"), user_notes_path(@user) %>
            </li>
            <li>
              <%= link_to t(".traces"), :controller => "traces", :action => "index", :display_name => @user.display_name %>
@@@ -180,56 -180,60 +180,60 @@@
  
  <% end %>
  
+ <div class="row">
    <% if current_user and @user.id == current_user.id %>
-     <% if @user.home_lat.nil? or @user.home_lon.nil? %>
-       <div id="map" class="content_map">
-         <p id="no_home_location"><%= t(".if_set_location_html", :settings_link => (link_to t(".settings_link_text"), :controller => "users", :action => "account", :display_name => current_user.display_name)) %></p>
-       </div>
-     <% else %>
-       <% content_for :head do %>
-         <%= javascript_include_tag "user" %>
+     <div class="col-md order-md-last">
+       <% if @user.home_lat.nil? or @user.home_lon.nil? %>
+         <div id="map" class="content_map">
+           <p id="no_home_location"><%= t(".if_set_location_html", :settings_link => (link_to t(".settings_link_text"), :controller => "users", :action => "account", :display_name => current_user.display_name)) %></p>
+         </div>
+       <% else %>
+         <% content_for :head do %>
+           <%= javascript_include_tag "user" %>
+         <% end %>
+         <% user_data = {
+              :lon => current_user.home_lon,
+              :lat => current_user.home_lat,
+              :icon => image_path("marker-red.png"),
+              :description => render(:partial => "popup", :object => current_user, :locals => { :type => "your location" })
+            } %>
+         <%= tag.div "", :id => "map", :class => "content_map", :data => { :user => user_data } %>
        <% end %>
-       <% user_data = {
-            :lon => current_user.home_lon,
-            :lat => current_user.home_lat,
-            :icon => image_path("marker-red.png"),
-            :description => render(:partial => "popup", :object => current_user, :locals => { :type => "your location" })
-          } %>
-       <%= tag.div "", :id => "map", :class => "content_map", :data => { :user => user_data } %>
-     <% end %>
- 
-     <% friends = @user.friends %>
-     <% nearby = @user.nearby - friends %>
- 
-   <div class="activity-block column-1">
-     <h3><%= t ".my friends" %></h3>
- 
-     <% if friends.empty? %>
-       <%= t ".no friends" %>
-     <% else %>
-       <ul class='secondary-actions clearfix'>
-         <li><%= link_to t(".friends_changesets"), friend_changesets_path %></li>
-         <li><%= link_to t(".friends_diaries"), friends_diary_entries_path %></li>
-       </ul>
-       <div id="friends-container">
-         <%= render :partial => "contact", :collection => friends, :locals => { :type => "friend" } %>
-       </div>
-     <% end %>
-   </div>
  
-   <div class="activity-block column-1">
-     <h3><%= t ".nearby users" %></h3>
- 
-     <% if nearby.empty? %>
-       <%= t ".no nearby users" %>
-     <% else %>
-       <ul class='secondary-actions clearfix'>
-         <li><%= link_to t(".nearby_changesets"), nearby_changesets_path %></li>
-         <li><%= link_to t(".nearby_diaries"), nearby_diary_entries_path %></li>
-       </ul>
-       <div id="nearbyusers">
-         <%= render :partial => "contact", :collection => nearby, :locals => { :type => "nearby mapper" } %>
-       </div>
-     <% end %>
-   </div>
- <% end %>
+       <% friends = @user.friends %>
+       <% nearby = @user.nearby - friends %>
+     </div>
+ 
+     <div class="col-md">
+       <h3><%= t ".my friends" %></h3>
+ 
+       <% if friends.empty? %>
+         <%= t ".no friends" %>
+       <% else %>
+         <ul class='secondary-actions clearfix'>
+           <li><%= link_to t(".friends_changesets"), friend_changesets_path %></li>
+           <li><%= link_to t(".friends_diaries"), friends_diary_entries_path %></li>
+         </ul>
+         <div id="friends-container">
+           <%= render :partial => "contact", :collection => friends, :locals => { :type => "friend" } %>
+         </div>
+       <% end %>
+ 
+       <hr>
+ 
+       <h3><%= t ".nearby users" %></h3>
+ 
+       <% if nearby.empty? %>
+         <%= t ".no nearby users" %>
+       <% else %>
+         <ul class='secondary-actions clearfix'>
+           <li><%= link_to t(".nearby_changesets"), nearby_changesets_path %></li>
+           <li><%= link_to t(".nearby_diaries"), nearby_diary_entries_path %></li>
+         </ul>
+         <div id="nearbyusers">
+           <%= render :partial => "contact", :collection => nearby, :locals => { :type => "nearby mapper" } %>
+         </div>
+       <% end %>
+     </div>
+   <% end %>
+ </div>