From: John Firebaugh <john.firebaugh@gmail.com>
Date: Tue, 3 Dec 2013 22:07:33 +0000 (-0800)
Subject: Use a table for tags
X-Git-Tag: live~4561
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/a6fb1ead5b9dc773813526c2d3e3beea0daebe17

Use a table for tags

Fixes #595
---

diff --git a/app/assets/stylesheets/common.css.scss b/app/assets/stylesheets/common.css.scss
index 4ec0dced3..75c921336 100644
--- a/app/assets/stylesheets/common.css.scss
+++ b/app/assets/stylesheets/common.css.scss
@@ -1069,20 +1069,20 @@ header .search_form {
     border: 1px solid #ddd;
     border-radius: 3px;
     font-size: 12px;
+    table-layout: fixed;
+    border-collapse: separate;
 
-    li {
+    th, td {
       border-bottom: 1px solid #ddd;
     }
 
-    li:last-child {
+    tr:last-child th, tr:last-child td {
       border-bottom: 0;
     }
 
     .browse-tag-k,
     .browse-tag-v {
-      display: inline-block;
       width: 50%;
-      float: left;
       padding: 6px 10px;
       word-wrap: break-word;
     }
diff --git a/app/views/browse/_tag.html.erb b/app/views/browse/_tag.html.erb
index b9296fa48..2081ba28d 100644
--- a/app/views/browse/_tag.html.erb
+++ b/app/views/browse/_tag.html.erb
@@ -1,4 +1,4 @@
-<li class='clearfix'>
-  <span class='browse-tag-k'><%= format_key(tag[0]) %></span>
-  <span class='browse-tag-v'><%= format_value(tag[0], tag[1]) %></span>
-</li>
+<tr>
+  <th class='browse-tag-k'><%= format_key(tag[0]) %></th>
+  <td class='browse-tag-v'><%= format_value(tag[0], tag[1]) %></td>
+</tr>
diff --git a/app/views/browse/_tag_details.html.erb b/app/views/browse/_tag_details.html.erb
index 090a23e7c..16e3b51b2 100644
--- a/app/views/browse/_tag_details.html.erb
+++ b/app/views/browse/_tag_details.html.erb
@@ -1,6 +1,6 @@
 <% unless tag_details.empty? %>
   <h4><%= t 'browse.tag_details.tags' %></h4>
-  <ul class='browse-tag-list'>
+  <table class='browse-tag-list'>
     <%= render :partial => "tag", :collection => tag_details.sort %>
-  </ul>
+  </table>
 <% end %>