From 191c767bac07413044bf50b119417dd1ad3ddff7 Mon Sep 17 00:00:00 2001
From: Adam Cooper <adam@networkpie.co.uk>
Date: Wed, 21 Jan 2015 10:56:00 +0000
Subject: [PATCH] Move CSS into separate file, fix width of plugins that use
 tables and adds table-responsive class for smaller screens.

Closes #755
---
 PHPCI/View/layout.phtml                       | 47 +---------------
 public/assets/css/AdminLTE-custom.css         | 53 +++++++++++++++++++
 public/assets/js/build-plugins/phpcpd.js      |  4 +-
 public/assets/js/build-plugins/phpcs.js       |  4 +-
 public/assets/js/build-plugins/phpdoccheck.js |  4 +-
 public/assets/js/build-plugins/phpmd.js       |  4 +-
 public/assets/js/build-plugins/phptallint.js  |  4 +-
 public/assets/js/build-plugins/phpunit.js     |  4 +-
 8 files changed, 66 insertions(+), 58 deletions(-)
 create mode 100644 public/assets/css/AdminLTE-custom.css

diff --git a/PHPCI/View/layout.phtml b/PHPCI/View/layout.phtml
index 1aca09c1..3375d9df 100644
--- a/PHPCI/View/layout.phtml
+++ b/PHPCI/View/layout.phtml
@@ -16,6 +16,7 @@
 
     <!-- Theme style -->
     <link href="<?php print PHPCI_URL; ?>assets/css/AdminLTE.css" rel="stylesheet" type="text/css" />
+    <link href="<?php print PHPCI_URL; ?>assets/css/AdminLTE-custom.css" rel="stylesheet" type="text/css" />
 
     <script>
         var PHPCI_URL = '<?php print PHPCI_URL; ?>';
@@ -41,53 +42,7 @@
     <![endif]-->
 
     <style>
-        .skin-blue .logo, .skin-blue .logo:hover  {
-            background-image: url('/assets/img/logo-large.png');
-            background-repeat: no-repeat;
-            background-size: 40%;
-            background-position: 65px;
-            text-indent: -5000px;
-        }
 
-        .build-info-panel {
-
-        }
-
-            .build-info-panel .box-header h1.box-title {
-                border: 0;
-                font-size: 1.5em;
-                font-weight: bold;
-                margin-left: 110px;
-            }
-
-            .build-info-panel h1.box-title span {
-                font-weight: normal;
-            }
-
-            .build-info-panel img {
-                border: 2px solid #fff;
-                border-radius: 50%;
-                margin-top: -40px;
-            }
-
-            .build-info-panel #build-info {
-                margin-left: 110px;
-                min-height: 50px;
-            }
-
-            .build-info-panel .commit-message {
-                margin-bottom: 20px;
-            }
-
-        .small-box h3 a, .small-box h4 a {
-            color: #fff;
-        }
-
-        .pagination>li>span {
-            font-weight: bold;
-            background: #337ab7;
-            color: #fff;
-        }
 
     </style>
 </head>
diff --git a/public/assets/css/AdminLTE-custom.css b/public/assets/css/AdminLTE-custom.css
new file mode 100644
index 00000000..009a360b
--- /dev/null
+++ b/public/assets/css/AdminLTE-custom.css
@@ -0,0 +1,53 @@
+.skin-blue .logo, .skin-blue .logo:hover  {
+    background-image: url('/assets/img/logo-large.png');
+    background-repeat: no-repeat;
+    background-size: 40%;
+    background-position: 65px;
+    text-indent: -5000px;
+}
+
+.build-info-panel {
+
+}
+
+    .build-info-panel .box-header h1.box-title {
+        border: 0;
+        font-size: 1.5em;
+        font-weight: bold;
+        margin-left: 110px;
+    }
+
+    .build-info-panel h1.box-title span {
+        font-weight: normal;
+    }
+
+    .build-info-panel img {
+        border: 2px solid #fff;
+        border-radius: 50%;
+        margin-top: -40px;
+    }
+
+    .build-info-panel #build-info {
+        margin-left: 110px;
+        min-height: 50px;
+    }
+
+    .build-info-panel .commit-message {
+        margin-bottom: 20px;
+    }
+
+.small-box h3 a, .small-box h4 a {
+    color: #fff;
+}
+
+.pagination>li>span {
+    font-weight: bold;
+    background: #337ab7;
+    color: #fff;
+}
+
+#plugins table td {
+    max-width: 300px;
+    overflow: hidden;
+    text-overflow: ellipsis;
+}
diff --git a/public/assets/js/build-plugins/phpcpd.js b/public/assets/js/build-plugins/phpcpd.js
index c9733678..2a0f3e16 100644
--- a/public/assets/js/build-plugins/phpcpd.js
+++ b/public/assets/js/build-plugins/phpcpd.js
@@ -23,14 +23,14 @@ var phpcpdPlugin = ActiveBuild.UiPlugin.extend({
 
     render: function() {
 
-        return $('<table class="table" id="phpcpd-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phpcpd-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>'+Lang.get('file')+'</th>' +
             '   <th>'+Lang.get('start')+'</th>' +
             '   <th>'+Lang.get('end')+'</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
 
     },
 
diff --git a/public/assets/js/build-plugins/phpcs.js b/public/assets/js/build-plugins/phpcs.js
index f3b1fe67..3e71ba72 100644
--- a/public/assets/js/build-plugins/phpcs.js
+++ b/public/assets/js/build-plugins/phpcs.js
@@ -22,14 +22,14 @@ var phpcsPlugin = ActiveBuild.UiPlugin.extend({
     },
 
     render: function() {
-        return $('<table class="table" id="phpcs-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phpcs-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>'+Lang.get('file')+'</th>' +
             '   <th>'+Lang.get('line')+'</th>' +
             '   <th>'+Lang.get('message')+'</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
     },
 
     onUpdate: function(e) {
diff --git a/public/assets/js/build-plugins/phpdoccheck.js b/public/assets/js/build-plugins/phpdoccheck.js
index 7d284617..04a6a0ad 100644
--- a/public/assets/js/build-plugins/phpdoccheck.js
+++ b/public/assets/js/build-plugins/phpdoccheck.js
@@ -24,7 +24,7 @@ var phpdoccheckPlugin = ActiveBuild.UiPlugin.extend({
     },
 
     render: function() {
-        return $('<table class="table" id="phpdoccheck-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phpdoccheck-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>'+Lang.get('file')+'</th>' +
@@ -32,7 +32,7 @@ var phpdoccheckPlugin = ActiveBuild.UiPlugin.extend({
             '   <th>'+Lang.get('class')+'</th>' +
             '   <th>'+Lang.get('method')+'</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
     },
 
     onUpdate: function(e) {
diff --git a/public/assets/js/build-plugins/phpmd.js b/public/assets/js/build-plugins/phpmd.js
index 57723ba9..d6d5f867 100644
--- a/public/assets/js/build-plugins/phpmd.js
+++ b/public/assets/js/build-plugins/phpmd.js
@@ -25,7 +25,7 @@ var phpmdPlugin = ActiveBuild.UiPlugin.extend({
 
     render: function() {
 
-        return $('<table class="table" id="phpmd-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phpmd-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>'+Lang.get('file')+'</th>' +
@@ -33,7 +33,7 @@ var phpmdPlugin = ActiveBuild.UiPlugin.extend({
             '   <th>'+Lang.get('end')+'</th>' +
             '   <th>'+Lang.get('message')+'</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
     },
 
     onUpdate: function(e) {
diff --git a/public/assets/js/build-plugins/phptallint.js b/public/assets/js/build-plugins/phptallint.js
index 9d981c53..fe549b99 100644
--- a/public/assets/js/build-plugins/phptallint.js
+++ b/public/assets/js/build-plugins/phptallint.js
@@ -22,14 +22,14 @@ var phptalPlugin = ActiveBuild.UiPlugin.extend({
     },
 
     render: function() {
-        return $('<table class="table" id="phptal-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phptal-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>File</th>' +
             '   <th>Line</th>' +
             '   <th>Message</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
     },
 
     onUpdate: function(e) {
diff --git a/public/assets/js/build-plugins/phpunit.js b/public/assets/js/build-plugins/phpunit.js
index 0e648e9d..4a55d905 100644
--- a/public/assets/js/build-plugins/phpunit.js
+++ b/public/assets/js/build-plugins/phpunit.js
@@ -25,12 +25,12 @@ var phpunitPlugin = ActiveBuild.UiPlugin.extend({
 
     render: function() {
 
-        return $('<table class="table" id="phpunit-data">' +
+        return $('<div class="table-responsive"><table class="table" id="phpunit-data">' +
             '<thead>' +
             '<tr>' +
             '   <th>'+Lang.get('test')+'</th>' +
             '</tr>' +
-            '</thead><tbody></tbody></table>');
+            '</thead><tbody></tbody></table></div>');
     },
 
     onUpdate: function(e) {