Merge pull request javierbyte#16 from Need4Video/patch-1

javierbyte · javierbyte · commit 5615587807e0 · 2015-11-11T10:57:05.000-06:00
Bug fixed &amp; compression optimized using CSS color names
diff --git a/src/App.js b/src/App.js
@@ -9,11 +9,18 @@ var base64ImageToRGBArray = require('./lib/base64ImageToRGBArray')
 function compressColor (rgb) {
   var hex = tinycolor(rgb).toHexString()
 
-  if (hex[1] === hex[2] && hex[3] === hex[4] && hex[5] === hex[6]) {
-    return '#' + hex[1] + hex[3] + hex[4]
-  } else {
-    return hex
+  switch (hex) { // based on CSS3 supported color names http://www.w3.org/TR/css3-color/
+    case '#c0c0c0': return 'silver';
+    case '#808080': return 'gray';
+    case '#800000': return 'maroon';
+    case '#ff0000': return 'red';
+    case '#800080': return 'purple';
+    case '#008000': return 'green';
+    case '#808000': return 'olive';
+    case '#000080': return 'navy';
+    case '#008080': return 'teal';
   }
+  return hex[1] === hex[2] && hex[3] === hex[4] && hex[5] === hex[6] ? "#" + hex[1] + hex[3] + hex[5] : hex;
 }
 
 export const App = React.createClass({
