From 7a65ee328286f321d5a805c7abc23a8c79f251c2 Mon Sep 17 00:00:00 2001
From: Jeff Atwood <jatwood@discourse.org>
Date: Thu, 26 Jun 2014 00:08:23 -0700
Subject: [PATCH] convert topic summary links to <table>

---
 .../templates/components/topic-map.js.handlebars  | 13 +++++++++----
 app/assets/stylesheets/desktop/topic-post.scss    | 15 ++++-----------
 app/assets/stylesheets/mobile/topic-post.scss     |  6 +-----
 3 files changed, 14 insertions(+), 20 deletions(-)

diff --git a/app/assets/javascripts/discourse/templates/components/topic-map.js.handlebars b/app/assets/javascripts/discourse/templates/components/topic-map.js.handlebars
index e52616147..4675c672d 100644
--- a/app/assets/javascripts/discourse/templates/components/topic-map.js.handlebars
+++ b/app/assets/javascripts/discourse/templates/components/topic-map.js.handlebars
@@ -57,17 +57,22 @@
 
   {{#if infoLinks}}
     <section class='links'>
-        <ul class='topic-links'>
+
+        <table class='topic-links'>
           {{#groupedEach infoLinks}}
-            <li>
+            <tr>
+            <td>
               <span class='badge badge-notification clicks' title='{{i18n topic_map.clicks count=clicks}}'>{{clicks}}</span>
+            </td>
+            <td>
               <a href="{{unbound url}}" target="_blank" class='topic-link track-link' data-user-id="{{unbound user_id}}" data-ignore-post-id="true" title="{{unbound url}}">
                 {{#if title}}{{title}}{{else}}{{shorten-url url}}{{/if}}
               </a>
               {{link-domain this}}
-            </li>
+            </td>
+            </tr>
           {{/groupedEach}}
-        </ul>
+        </table>
 
         {{#if showAllLinksControls}}
           <div class='link-summary'>
diff --git a/app/assets/stylesheets/desktop/topic-post.scss b/app/assets/stylesheets/desktop/topic-post.scss
index e51841b20..ad03c857d 100644
--- a/app/assets/stylesheets/desktop/topic-post.scss
+++ b/app/assets/stylesheets/desktop/topic-post.scss
@@ -342,10 +342,6 @@ a.star {
     &:hover {border: 1px solid $primary !important;}
   }
 
-  .topic-links li {
-    word-wrap: break-word;
-  }
-
   h3 {
     margin-bottom: 4px;
     color: $primary;
@@ -428,16 +424,13 @@ a.star {
     border-top: 2px solid scale-color-diff();
   }
 
-.participants { // PMs //
-  .user {float: left; margin: 7px 20px 7px 0;}
-}
+  .participants { // PMs //
+    .user {float: left; margin: 7px 20px 7px 0;}
+  }
 
   .topic-links {
-    li > a {
-      margin-left: 5px;
-    }
     .badge-notification {
-      margin: 0 0 4px;
+      margin-right: 5px;
     }
   }
   .buttons {
diff --git a/app/assets/stylesheets/mobile/topic-post.scss b/app/assets/stylesheets/mobile/topic-post.scss
index 17677d20f..963692277 100644
--- a/app/assets/stylesheets/mobile/topic-post.scss
+++ b/app/assets/stylesheets/mobile/topic-post.scss
@@ -165,10 +165,6 @@ a.star {
     border-radius: 0 4px 0 4px;
   }
 
-  .topic-links li {
-    word-wrap: break-word;
-  }
-
   margin: 30px 10px 10px 10px;
   background: scale-color(scale-color-diff(), $lightness: 40%);
 
@@ -259,7 +255,7 @@ a.star {
 
   .topic-links {
     .badge-notification {
-      margin: 0 0 4px;
+      margin-right: 5px;
     }
   }
   .buttons {