From ff3dc13f90da40e94aac117402648cc336ba72ad Mon Sep 17 00:00:00 2001
From: "Luke \"Jared\" Bennett" <lbennett@gitlab.com>
Date: Fri, 15 Jul 2016 19:46:26 +0100
Subject: [PATCH] Review changes

---
 .../javascripts/abuse_reports.js.coffee       | 30 ++++++++++---------
 app/assets/stylesheets/pages/admin.scss       | 13 +++++---
 2 files changed, 25 insertions(+), 18 deletions(-)

diff --git a/app/assets/javascripts/abuse_reports.js.coffee b/app/assets/javascripts/abuse_reports.js.coffee
index de768b12ef9..e9bd854044a 100644
--- a/app/assets/javascripts/abuse_reports.js.coffee
+++ b/app/assets/javascripts/abuse_reports.js.coffee
@@ -1,26 +1,28 @@
 class @AbuseReports
   MAX_MESSAGE_LENGTH = 300
-  MESSAGE_CELL_SELECTOR = 'table tbody tr td:nth-child(3)'
+  MESSAGE_CELL_SELECTOR = '.abuse-reports td:nth-child(3)'
 
   constructor: ->
     $(MESSAGE_CELL_SELECTOR).each @truncateLongMessage
-    $(document).on 'click', "#{MESSAGE_CELL_SELECTOR}", @toggleMessageTruncation
+    $(document)
+      .off 'click', MESSAGE_CELL_SELECTOR
+      .on 'click', MESSAGE_CELL_SELECTOR, @toggleMessageTruncation
 
   truncateLongMessage: ->
-    messageCellElement = $(this)
-    reportMessage = messageCellElement.text()
+    $messageCellElement = $(this)
+    reportMessage = $messageCellElement.text()
     if reportMessage.length > MAX_MESSAGE_LENGTH
-      messageCellElement.attr 'data-original-message', reportMessage
-      messageCellElement.attr 'data-message-truncated', 'true'
-      messageCellElement.text "#{reportMessage.substr 0, MAX_MESSAGE_LENGTH}..."
+      $messageCellElement.data 'original-message', reportMessage
+      $messageCellElement.data 'message-truncated', 'true'
+      $messageCellElement.text "#{reportMessage.substr 0, MAX_MESSAGE_LENGTH}..."
 
   toggleMessageTruncation: ->
-    messageCellElement = $(this)
-    originalMessage = messageCellElement.attr 'data-original-message'
+    $messageCellElement = $(this)
+    originalMessage = $messageCellElement.data 'original-message'
     return if not originalMessage
-    if messageCellElement.attr('data-message-truncated') is 'true'
-      messageCellElement.attr 'data-message-truncated', 'false'
-      messageCellElement.text originalMessage
+    if $messageCellElement.data('message-truncated') is 'true'
+      $messageCellElement.data 'message-truncated', 'false'
+      $messageCellElement.text originalMessage
     else
-      messageCellElement.attr 'data-message-truncated', 'true'
-      messageCellElement.text "#{originalMessage.substr 0, MAX_MESSAGE_LENGTH}..."
+      $messageCellElement.data 'message-truncated', 'true'
+      $messageCellElement.text "#{originalMessage.substr 0, MAX_MESSAGE_LENGTH}..."
diff --git a/app/assets/stylesheets/pages/admin.scss b/app/assets/stylesheets/pages/admin.scss
index dd3df93a200..4ea8292ec2a 100644
--- a/app/assets/stylesheets/pages/admin.scss
+++ b/app/assets/stylesheets/pages/admin.scss
@@ -100,10 +100,15 @@
 
 .abuse-reports {
   table-layout: fixed;
-  td {
-    vertical-align: top !important;
-    &.action-buttons a {
-      white-space: normal;
+  .action-buttons a {
+    white-space: normal;
+  }
+  @media (min-width: $screen-sm-min) {
+    th {
+      width: 15%;
+      &.wide {
+        width: 55%;
+      }
     }
   }
 }
-- 
GitLab