Merge pull request #217 from alexcorre/colors-tweaks

Colors tweaks

lib/components/BlameLine.js

@@ -11,21 +11,21 @@ function lastWord(str) {
 }
 
 function stringToColour(str) {
-  if(colours[str]) {
+  if (colours[str]) {
     return colours[str];
-  } else {
-    var hash = 0;
-    for (var i = 0; i < str.length; i++) {
-        hash = str.charCodeAt(i) + ((hash << 5) - hash);
-    }
-    var colour = '#';
-    for (var i = 0; i < 3; i++) {
-        var value = (hash >> (i * 8)) & 0xFF;
-        colour += ('00' + value.toString(16)).substr(-2);
-    }
-    colours[str] = colour;
-    return colour;
   }
+
+  let hash = 0;
+  for (let i = 0; i < str.length; i++) {              // eslint-disable-line no-plusplus
+    hash = str.charCodeAt(i) + ((hash << 5) - hash);  // eslint-disable-line no-bitwise
+  }
+  let colour = '#';
+  for (let i = 0; i < 3; i++) {                       // eslint-disable-line no-plusplus
+    var value = (hash >> (i * 8)) & 0xFF;             // eslint-disable-line no-bitwise
+    colour += (`00${value.toString(16)}`).substr(-2);
+  }
+  colours[str] = colour;
+  return colour;
 }
 
 export default function BlameLine(props) {
@@ -42,7 +42,7 @@ export default function BlameLine(props) {
 
   const displayName = showOnlyLastNames ? lastWord(author) : author;
   return (
-    <div className={`blame-line ${className}`} style={{ borderRight: colorCommitAuthors ? `thick solid ${stringToColour(author)}` : 'none' }}>
+    <div className={`blame-line ${className}`} style={{ borderRight: colorCommitAuthors ? `2px solid ${stringToColour(author)}` : 'none' }}>
       <a href={viewCommitUrl}>
         {showHash ? <span className="hash">{hash.substring(0, HASH_LENGTH)}</span> : null}
         <span className="date">{date}</span>

lib/config.js

@@ -29,8 +29,8 @@ export default {
     type: 'boolean',
     default: true,
   },
-  colorCommitAuthors : {
+  colorCommitAuthors: {
     type: 'boolean',
-    default: true
+    default: false,
   },
 };