From ea2a31d2d7653b3f1406c9ec9fe3e3a4d1f6151f Mon Sep 17 00:00:00 2001
From: Tim Roes <github@timroes.de>
Date: Tue, 8 Aug 2017 12:11:26 +0200
Subject: [PATCH] Move build-info to status page (#13341)

* Move build-info to status page
* Remove info property from management section
---
 .../kibana/public/management/index.js          |  4 +---
 .../kibana/public/management/landing.html      | 14 --------------
 .../kibana/public/management/styles/main.less  | 18 ------------------
 .../status_page/public/status_page.html        |  6 ++++++
 .../status_page/public/status_page.js          |  7 ++++++-
 .../status_page/public/status_page.less        |  4 ++++
 src/ui/public/management/section.js            |  1 -
 7 files changed, 17 insertions(+), 37 deletions(-)

diff --git a/src/core_plugins/kibana/public/management/index.js b/src/core_plugins/kibana/public/management/index.js
index 4f24af642723a..372189fbcb673 100644
--- a/src/core_plugins/kibana/public/management/index.js
+++ b/src/core_plugins/kibana/public/management/index.js
@@ -25,7 +25,7 @@ require('ui/index_patterns/route_setup/load_default')({
 
 uiModules
 .get('apps/management')
-.directive('kbnManagementApp', function (Private, $location, timefilter, buildNum, buildSha) {
+.directive('kbnManagementApp', function (Private, $location, timefilter) {
   return {
     restrict: 'E',
     template: appTemplate,
@@ -46,8 +46,6 @@ uiModules
           item.active = `#${$location.path()}`.indexOf(item.url) > -1;
         });
       }
-
-      management.getSection('kibana').info = `Build ${buildNum}, Commit SHA ${buildSha.substr(0, 8)}`;
     }
   };
 });
diff --git a/src/core_plugins/kibana/public/management/landing.html b/src/core_plugins/kibana/public/management/landing.html
index b487824b3d895..ee7e1c3266872 100644
--- a/src/core_plugins/kibana/public/management/landing.html
+++ b/src/core_plugins/kibana/public/management/landing.html
@@ -9,7 +9,6 @@
     <div
       ng-if="section.visibleItems.length > 0"
       ng-repeat="section in sections"
-      ng-class="{ 'management-section-info-expanded': section.showInfo }"
       class="page-row"
     >
       <div class="kuiPanel management-panel">
@@ -22,15 +21,6 @@
               {{::section.display}}
             </div>
           </div>
-
-          <div class="kuiPanelHeaderSection" ng-if="section.info">
-            <div class="kuiText">
-              <span
-                class="fa fa-info-circle management-panel__heading-icon--secondary"
-                ng-click="section.showInfo = !!!section.showInfo"
-              ></span>
-            </div>
-          </div>
         </div>
 
         <div class="kuiPanelBody management-panel__body">
@@ -55,10 +45,6 @@
               </li>
             </ul>
           </div>
-
-          <div class="management-section-info">
-            {{::section.info}}
-          </div>
         </div>
       </div>
     </div>
diff --git a/src/core_plugins/kibana/public/management/styles/main.less b/src/core_plugins/kibana/public/management/styles/main.less
index 8b70561a2e0fe..fab9177733212 100644
--- a/src/core_plugins/kibana/public/management/styles/main.less
+++ b/src/core_plugins/kibana/public/management/styles/main.less
@@ -55,20 +55,6 @@ kbn-management-landing {
     }
   }
 
-  .management-section-info {
-    bottom: 10px;
-    display: none;
-    color: @kibanaPink1;
-    font-size: 0.8em;
-    position: absolute;
-  }
-
-  .management-section-info-expanded {
-    .management-section-info {
-      display: block;
-    }
-  }
-
   /**
    * 1. Set icon size.
    * 2. Hack the panel title to move to the left a bit.
@@ -102,10 +88,6 @@ kbn-management-landing {
     background-image: url("~ui/icons/logstash-gray.svg");
   }
 
-  .management-panel__heading-icon--secondary {
-    font-size: 0.9em;
-    padding: 5px;
-  }
 }
 
 kbn-management-objects {
diff --git a/src/core_plugins/status_page/public/status_page.html b/src/core_plugins/status_page/public/status_page.html
index 6979eebd9c5be..a60f425b8b831 100644
--- a/src/core_plugins/status_page/public/status_page.html
+++ b/src/core_plugins/status_page/public/status_page.html
@@ -43,4 +43,10 @@ <h4 ng-if="!ui.statuses && !ui.loading" class="statuses_missing">
       </tr>
     </table>
   </div>
+
+  <footer class="row">
+    <div class="col-xs-12 text-right build-info">
+      Build {{::ui.buildInfo.num}}, Commit SHA {{::ui.buildInfo.sha}}
+    </div>
+  </footer>
 </div>
diff --git a/src/core_plugins/status_page/public/status_page.js b/src/core_plugins/status_page/public/status_page.js
index 413208065a5cd..2fba5d8ca659b 100644
--- a/src/core_plugins/status_page/public/status_page.js
+++ b/src/core_plugins/status_page/public/status_page.js
@@ -8,10 +8,15 @@ import { uiModules } from 'ui/modules';
 
 const chrome = require('ui/chrome')
 .setRootTemplate(require('plugins/status_page/status_page.html'))
-.setRootController('ui', function ($http) {
+.setRootController('ui', function ($http, buildNum, buildSha) {
   const ui = this;
   ui.loading = false;
 
+  ui.buildInfo = {
+    num: buildNum,
+    sha: buildSha.substr(0, 8)
+  };
+
   ui.refresh = function () {
     ui.loading = true;
 
diff --git a/src/core_plugins/status_page/public/status_page.less b/src/core_plugins/status_page/public/status_page.less
index 7b391d5477157..99894ccf1badd 100644
--- a/src/core_plugins/status_page/public/status_page.less
+++ b/src/core_plugins/status_page/public/status_page.less
@@ -178,3 +178,7 @@
 .overall_state_red {
   .state(@status-red, @icon-red);
 }
+
+.build-info {
+  color: #555;
+}
diff --git a/src/ui/public/management/section.js b/src/ui/public/management/section.js
index fb1a57ec875ce..0243f240181a4 100644
--- a/src/ui/public/management/section.js
+++ b/src/ui/public/management/section.js
@@ -9,7 +9,6 @@ export class ManagementSection {
    * @param {number|null} options.order
    * @param {string|null} options.display - defaults to id
    * @param {string|null} options.url - defaults to ''
-   * @param {string|null} options.info
    * @param {boolean|null} options.visible - defaults to true
    * @param {boolean|null} options.disabled - defaults to false
    * @param {string|null} options.tooltip - defaults to ''