From 9163846c5a519dbe3310cdf83a9c36442d5779c8 Mon Sep 17 00:00:00 2001
From: David Kaltschmidt <david.kaltschmidt@gmail.com>
Date: Mon, 7 Mar 2016 13:44:29 +0100
Subject: [PATCH] Review feedback for node borders

---
 client/app/scripts/charts/node.js          | 1 +
 client/app/scripts/utils/contrast-utils.js | 6 +++---
 client/app/styles/main.less                | 6 +++++-
 3 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/client/app/scripts/charts/node.js b/client/app/scripts/charts/node.js
index 16f6a41d7..6a3cf4ebb 100644
--- a/client/app/scripts/charts/node.js
+++ b/client/app/scripts/charts/node.js
@@ -84,6 +84,7 @@ export default class Node extends React.Component {
       classNames.push('pseudo');
     }
     if (this.props.shape === 'cloud') {
+      // need to identify cloud in CSS because it's stroke is not set via CSS
       classNames.push('cloud');
     }
 
diff --git a/client/app/scripts/utils/contrast-utils.js b/client/app/scripts/utils/contrast-utils.js
index 60ca1be9e..e7f4c7a46 100644
--- a/client/app/scripts/utils/contrast-utils.js
+++ b/client/app/scripts/utils/contrast-utils.js
@@ -1,7 +1,7 @@
-const contrastMode = window.location.pathname.indexOf('contrast') > -1;
+export const contrastModeUrl = 'contrast.html';
+
+const contrastMode = window.location.pathname.indexOf(contrastModeUrl) > -1;
 
 export function isContrastMode() {
   return contrastMode;
 }
-
-export const contrastModeUrl = 'contrast.html';
diff --git a/client/app/styles/main.less b/client/app/styles/main.less
index 78318f297..dbb416c2e 100644
--- a/client/app/styles/main.less
+++ b/client/app/styles/main.less
@@ -328,7 +328,6 @@ h2 {
     }
 
     &.pseudo {
-      opacity: @node-pseudo-opacity;
       cursor: default;
 
       .node-label {
@@ -339,7 +338,12 @@ h2 {
         fill: @text-tertiary-color;
       }
 
+      .node {
+        opacity: @node-pseudo-opacity;
+      }
+
       .border {
+        opacity: @node-pseudo-opacity;
         stroke: @text-tertiary-color;
       }
     }