@samitouri / QOS-React-2 / commits / 66cdba3dfb

Avoid leading spaces when first argument to joinClasses is falsy.

This detail is going to become more important once the idiom `className={joinClasses(this.props.className, newClass)}` becomes more common, as it will when we move away from `this.transferPropsTo`.

Ben Newman committed Jul 8, 2014 at 14:27 UTC 66cdba3dfbc3cd71d891b805a35f592d05a684d0
1 file changed +3 -1
src/utils/joinClasses.js
+3 -1
@@ -35,7 +35,9 @@ function joinClasses(className/*, ... */) {
35 if (argLength > 1) {
36 for (var ii = 1; ii < argLength; ii++) {
37 nextClass = arguments[ii];
38 - nextClass && (className += ' ' + nextClass);
38 + if (nextClass) {
39 + className = (className ? className + ' ' : '') + nextClass;
40 + }
41 }
42 }
43 return className;