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

[rails.git] / app / views / traces / show.html.erb

diff --git a/app/views/traces/show.html.erb b/app/views/traces/show.html.erb
index a8f9c5967d67f4bffa891e719c7fdc27567ab2d5..51b21f9828e0f72cfdd56488f0e6657e9098a442 100644 (file)
--- a/app/views/traces/show.html.erb
+++ b/app/views/traces/show.html.erb
@@ -4,7 +4,7 @@
 
 <% if Settings.status != "gpx_offline" %>
   <% if @trace.inserted %>
 
 <% if Settings.status != "gpx_offline" %>
   <% if @trace.inserted %>

-    <img src="<%= url_for :controller => "traces", :action => "picture", :id => @trace.id, :display_name => @trace.user.display_name %>">
+    <%= image_tag trace_picture_path(@trace.user, @trace), :class => "trace_image" %>

   <% else %>
     <span class="text-danger"><%= t ".pending" %></span>
   <% end %>
   <% else %>
     <span class="text-danger"><%= t ".pending" %></span>
   <% end %>


@@ -12,38 +12,38 @@
 
 <table class="table table-borderless table-sm">
   <tr>
 
 <table class="table table-borderless table-sm">
   <tr>

-    <td><%= t ".filename" %></td>
+    <th><%= t ".filename" %></th>

     <td><%= @trace.name %> (<%= link_to t(".download"), trace_data_path(@trace) %>)</td>
   </tr>
   <tr>
     <td><%= @trace.name %> (<%= link_to t(".download"), trace_data_path(@trace) %>)</td>
   </tr>
   <tr>

-    <td><%= t ".uploaded" %></td>
+    <th><%= t ".uploaded" %></th>

     <td><%= l @trace.timestamp, :format => :friendly %></td>
   </tr>
   <% if @trace.inserted? %>
   <tr>
     <td><%= l @trace.timestamp, :format => :friendly %></td>
   </tr>
   <% if @trace.inserted? %>
   <tr>

-    <td><%= t ".points" %></td>
+    <th><%= t ".points" %></th>

     <td><%= number_with_delimiter(@trace.size) %></td></tr>
   <tr>
     <td><%= number_with_delimiter(@trace.size) %></td></tr>
   <tr>

-    <td><%= t ".start_coordinates" %></td>
+    <th><%= t ".start_coordinates" %></th>

     <td>
       <div class="d-inline">
         <%= t ".coordinates_html",
               :latitude => tag.span(number_with_delimiter(@trace.latitude), :class => "latitude"),
               :longitude => tag.span(number_with_delimiter(@trace.longitude), :class => "longitude") %>
       </div>
     <td>
       <div class="d-inline">
         <%= t ".coordinates_html",
               :latitude => tag.span(number_with_delimiter(@trace.latitude), :class => "latitude"),
               :longitude => tag.span(number_with_delimiter(@trace.longitude), :class => "longitude") %>
       </div>

-      (<%= link_to t(".map"), :controller => "site", :action => "index", :mlat => @trace.latitude, :mlon => @trace.longitude, :anchor => "map=14/#{@trace.latitude}/#{@trace.longitude}" %> / <%= link_to t(".edit"), :controller => "site", :action => "edit", :gpx => @trace.id, :anchor => "map=14/#{@trace.latitude}/#{@trace.longitude}" %>)</td>
+      (<%= link_to t(".map"), root_path(:mlat => @trace.latitude, :mlon => @trace.longitude, :anchor => "map=14/#{@trace.latitude}/#{@trace.longitude}") %> / <%= link_to t(".edit"), edit_path(:gpx => @trace.id, :anchor => "map=14/#{@trace.latitude}/#{@trace.longitude}") %>)</td>

   </tr>
   <% end %>
   <tr>
   </tr>
   <% end %>
   <tr>

-    <td><%= t ".owner" %></td>
-    <td><%= link_to @trace.user.display_name, user_path(@trace.user) %></td>
+    <th><%= t ".owner" %></th>
+    <td><%= link_to @trace.user.display_name, @trace.user %></td>

   </tr>
   <tr>
   </tr>
   <tr>

-    <td><%= t ".description" %></td>
+    <th><%= t ".description" %></th>

     <td><%= @trace.description %></td>
   </tr>
   <tr>
     <td><%= @trace.description %></td>
   </tr>
   <tr>

-    <td><%= t ".tags" %></td>
+    <th><%= t ".tags" %></th>

     <td>
     <% unless @trace.tags.empty? %>
       <%= safe_join(@trace.tags.collect { |tag| link_to tag.tag, :controller => "traces", :action => "index", :tag => tag.tag, :id => nil }, ", ") %>
     <td>
     <% unless @trace.tags.empty? %>
       <%= safe_join(@trace.tags.collect { |tag| link_to tag.tag, :controller => "traces", :action => "index", :tag => tag.tag, :id => nil }, ", ") %>


@@ -53,7 +53,7 @@
     </td>
   </tr>
   <tr>
     </td>
   </tr>
   <tr>

-    <td><%= t ".visibility" %></td>
+    <th><%= t ".visibility" %></th>

     <td><%= t "traces.visibility.#{@trace.visibility}" %></td>
   </tr>
 </table>
     <td><%= t "traces.visibility.#{@trace.visibility}" %></td>
   </tr>
 </table>


@@ -63,6 +63,6 @@
     <% if current_user == @trace.user %>
       <%= link_to t(".edit_trace"), edit_trace_path(@trace), :class => "btn btn-outline-primary" %>
     <% end %>
     <% if current_user == @trace.user %>
       <%= link_to t(".edit_trace"), edit_trace_path(@trace), :class => "btn btn-outline-primary" %>
     <% end %>

-    <%= link_to t(".delete_trace"), { :controller => "traces", :action => "destroy", :id => @trace.id }, { :method => :delete, :class => "btn btn-outline-danger", :data => { :confirm => t(".confirm_delete") } } %>
+    <%= link_to t(".delete_trace"), @trace, { :method => :delete, :class => "btn btn-outline-danger", :data => { :confirm => t(".confirm_delete") } } %>

   </div>
 <% end %>
   </div>
 <% end %>