Merge pull request #3 from beekeeper-studio/fix/isNaN

use Number.isNaN as it's more robust

Author: azmy60

src/js/modules/Format/defaults/formatters/money.js

@@ -10,7 +10,7 @@ export default function(cell, formatterParams, onRendered){
 	var after = !!formatterParams.symbolAfter;
 	var precision = typeof formatterParams.precision !== "undefined" ? formatterParams.precision : 2;
 
-	if(isNaN(floatVal)){
+	if(Number.isNaN(floatVal)){
 		return this.emptyToSpace(this.sanitizeHTML(cell.getValue()));
 	}
 
@@ -41,4 +41,4 @@ export default function(cell, formatterParams, onRendered){
 	}else{
 		return after ? sign + value + symbol : sign + symbol + value;
 	}
-}
+}

src/js/modules/Format/defaults/formatters/star.js

@@ -17,7 +17,7 @@ export default function(cell, formatterParams, onRendered){
 	star.setAttribute("xml:space", "preserve");
 	star.style.padding = "0 1px";
 
-	value = value && !isNaN(value) ? parseInt(value) : 0;
+	value = value && !Number.isNaN(value) ? parseInt(value) : 0;
 
 	value = Math.max(0, Math.min(value, maxStars));
 
@@ -35,4 +35,4 @@ export default function(cell, formatterParams, onRendered){
 	element.setAttribute("aria-label", value);
 
 	return stars;
-}
+}

src/js/modules/Format/defaults/formatters/traffic.js

@@ -7,7 +7,7 @@ export default function(cell, formatterParams, onRendered){
 	color = "#666666",
 	percent, percentValue;
 
-	if(isNaN(value) || typeof cell.getValue() === "undefined"){
+	if(Number.isNaN(value) || typeof cell.getValue() === "undefined"){
 		return;
 	}
 
@@ -44,4 +44,4 @@ export default function(cell, formatterParams, onRendered){
 	el.style.backgroundColor = color;
 
 	return el;
-}
+}