Merge pull request #1280 from weaveworks/1271-broken-plugin-status

Nice and red and blinky ui label for broken plugins

client/app/scripts/components/plugins.js

@@ -1,10 +1,19 @@
 import React from 'react';
+import classNames from 'classnames';
 
 export default class Plugins extends React.Component {
-  renderPlugin(plugin) {
+  renderPlugin({id, label, description, status}) {
+    const error = status !== 'ok';
+    const className = classNames({ error });
+    const title = `Status: ${status} | Plugin description: ${description}`;
+
+    // Inner span to hold styling so we don't effect the "before:content"
     return (
-      <span className="plugins-plugin" key={plugin.id} title={plugin.description}>
-        {plugin.label || plugin.id}
+      <span className="plugins-plugin" key={id}>
+        <span className={className} title={title}>
+          {error && <span className="plugins-plugin-icon fa fa-exclamation-circle" />}
+          {label || id}
+        </span>
       </span>
     );
   }

client/app/scripts/stores/app-store.js

@@ -46,7 +46,7 @@ let highlightedEdgeIds = makeSet();
 let highlightedNodeIds = makeSet();
 let hostname = '...';
 let version = '...';
-let plugins = null;
+let plugins = [];
 let mouseOverEdgeId = null;
 let mouseOverNodeId = null;
 let nodeDetails = makeOrderedMap(); // nodeId -> details

client/app/styles/main.less

@@ -1007,6 +1007,18 @@ h2 {
     content: ', ';
   }
 
+  &-plugin-icon {
+    top: 1px;
+    position: relative;
+    font-size: 120%;
+    margin-right: 2px;
+  }
+
+  .error {
+    animation: blinking 2.0s 60 @base-ease; // blink for 2 minute;
+    color: @text-secondary-color;
+  }
+
   &-empty {
     opacity: @text-secondary-color;
   }