Merge pull request #22 from twitter-fabric/patch/shim-for-server-rendering

Shims out Velocity on the server

Author: fionawhim

lib/velocity-animate-shim.js

@@ -0,0 +1,11 @@
+// Shim to avoid requiring Velocity in Node environments, since it
+// requires window. Note that this just no-ops the components so
+// that they'll render, rather than doing something clever like
+// statically rendering the end state of any provided animations.
+if (typeof window !== 'undefined') {
+  module.exports = require('velocity-animate');
+} else {
+  var Velocity = function () {};
+  Velocity.velocityReactServerShim = true;
+  module.exports = Velocity;
+}

velocity-component.js

@@ -39,7 +39,7 @@ var _ = {
   omit: require('lodash/object/omit'),
 };
 var React = require('react');
-var Velocity = require('velocity-animate');
+var Velocity = require('./lib/velocity-animate-shim');
 
 var VelocityComponent = React.createClass({
   displayName: 'VelocityComponent',

velocity-helpers.js

@@ -3,7 +3,7 @@
 var _ = {
   isObject: require('lodash/lang/isObject'),
 };
-var Velocity = require('velocity-animate');
+var Velocity = require('./lib/velocity-animate-shim');
 
 var effectCounter = 0;
 

velocity-transition-group.js

@@ -40,7 +40,7 @@ var _ = {
   pluck: require('lodash/collection/pluck'),
 };
 var React = require('react/addons');
-var Velocity = require('velocity-animate');
+var Velocity = require('./lib/velocity-animate-shim');
 
 // Internal wrapper for the transitioned elements. Delegates all child lifecycle events to the
 // parent VelocityTransitionGroup so that it can co-ordinate animating all of the elements at once.
@@ -110,7 +110,7 @@ var VelocityTransitionGroup = React.createClass({
 
     // Without our custom childFactory, we just get a default TransitionGroup that doesn't do
     // anything special at all.
-    if (!this.constructor.disabledForTest) {
+    if (!this.constructor.disabledForTest && !Velocity.velocityReactServerShim) {
       transitionGroupProps.childFactory = this._wrapChild;
     }