From bfbb6faf5bfb65e5c2a509a2960b45b8650dfec3 Mon Sep 17 00:00:00 2001
From: Frank Elsinga <frank@elsinga.de>
Date: Wed, 18 Feb 2026 04:42:55 +0100
Subject: [PATCH] Move the tables on the traces page into an
 .table-resposponsive to work properly with too long content

---
 app/views/traces/_page.html.erb | 12 +++--
 app/views/traces/show.html.erb  | 94 +++++++++++++++++----------------
 2 files changed, 55 insertions(+), 51 deletions(-)

diff --git a/app/views/traces/_page.html.erb b/app/views/traces/_page.html.erb
index 422da5534..31a6f5f12 100644
--- a/app/views/traces/_page.html.erb
+++ b/app/views/traces/_page.html.erb
@@ -3,11 +3,13 @@
              :newer_id => @newer_traces_id,
              :older_id => @older_traces_id %>
 
-  <table id="trace_list" class="table table-borderless table-striped">
-    <tbody>
-      <%= render @traces %>
-    </tbody>
-  </table>
+  <div class="table-responsive">
+    <table id="trace_list" class="table table-borderless table-striped">
+      <tbody>
+        <%= render @traces %>
+      </tbody>
+    </table>
+  </div>
 
   <%= render "shared/pagination",
              :newer_id => @newer_traces_id,
diff --git a/app/views/traces/show.html.erb b/app/views/traces/show.html.erb
index 63adf1572..5f21cff36 100644
--- a/app/views/traces/show.html.erb
+++ b/app/views/traces/show.html.erb
@@ -10,53 +10,55 @@
   <% end %>
 <% end %>
 
-<table class="table table-borderless table-sm">
-  <tr>
-    <th><%= t ".filename" %></th>
-    <td><%= @trace.name %> (<%= link_to t(".download"), trace_data_path(@trace) %>)</td>
-  </tr>
-  <tr>
-    <th><%= t ".uploaded" %></th>
-    <td><%= l @trace.timestamp, :format => :friendly %></td>
-  </tr>
-  <% if @trace.inserted? %>
-  <tr>
-    <th><%= t ".points" %></th>
-    <td><%= number_with_delimiter(@trace.size) %></td></tr>
-  <tr>
-    <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>
-      (<%= 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>
-    <th><%= t ".owner" %></th>
-    <td><%= link_to @trace.user.display_name, @trace.user %></td>
-  </tr>
-  <tr>
-    <th><%= t ".description" %></th>
-    <td><%= @trace.description %></td>
-  </tr>
-  <tr>
-    <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 }, ", ") %>
-    <% else %>
-      <i><%= t ".none" %></i>
+<div class="table-responsive">
+  <table class="table table-borderless table-sm">
+    <tr>
+      <th><%= t ".filename" %></th>
+      <td><%= @trace.name %> (<%= link_to t(".download"), trace_data_path(@trace) %>)</td>
+    </tr>
+    <tr>
+      <th><%= t ".uploaded" %></th>
+      <td><%= l @trace.timestamp, :format => :friendly %></td>
+    </tr>
+    <% if @trace.inserted? %>
+    <tr>
+      <th><%= t ".points" %></th>
+      <td><%= number_with_delimiter(@trace.size) %></td></tr>
+    <tr>
+      <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>
+        (<%= 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 %>
-    </td>
-  </tr>
-  <tr>
-    <th><%= t ".visibility" %></th>
-    <td><%= t "traces.visibility.#{@trace.visibility}" %></td>
-  </tr>
-</table>
+    <tr>
+      <th><%= t ".owner" %></th>
+      <td><%= link_to @trace.user.display_name, @trace.user %></td>
+    </tr>
+    <tr>
+      <th><%= t ".description" %></th>
+      <td><%= @trace.description %></td>
+    </tr>
+    <tr>
+      <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 }, ", ") %>
+      <% else %>
+        <i><%= t ".none" %></i>
+      <% end %>
+      </td>
+    </tr>
+    <tr>
+      <th><%= t ".visibility" %></th>
+      <td><%= t "traces.visibility.#{@trace.visibility}" %></td>
+    </tr>
+  </table>
+</div>
 
 <% if current_user && (current_user==@trace.user || current_user.administrator? || current_user.moderator?) %>
   <div>
-- 
2.39.5