react-slick - npm Package Compare versions

Comparing version 0.3.8 to 0.3.9

lib/inner-slider.jsx

@@ -5,3 +5,3 @@ "use strict";
 var cloneWithProps = require('react/lib/cloneWithProps');
-var cx = require('react/lib/cx');
+var classnames = require('classnames');
 var EventHandlersMixin = require('./mixins/event-handlers');
@@ -28,9 +28,9 @@ var HelpersMixin = require('./mixins/helpers');
   renderDots: function () {
-    var classes, dotOptions;
+    var dotOptions;
     var dots = [];
     if (this.props.dots === true && this.state.slideCount > this.props.slidesToShow) {
       for (var i=0; i <= this.getDotCount(); i += 1) {
-        classes = {
+        var className = classnames({
           'slick-active': (this.state.currentSlide === i * this.props.slidesToScroll),
-        };
+        });
         dotOptions = {
@@ -40,3 +40,3 @@ message: 'index',
         };
-        dots.push(<li key={i} className={cx(classes)}><button onClick={this.changeSlide.bind(this, dotOptions)}>{i}</button></li>);
+        dots.push(<li key={i} className={className}><button onClick={this.changeSlide.bind(this, dotOptions)}>{i}</button></li>);
       }
@@ -133,4 +133,4 @@ return (
 
-      var prevArrow = <button key={0} ref='previous' type="button" data-role="none" className={cx(prevClasses)} style={{display: 'block'}} onClick={prevHandler}> Previous</button>;
-      var nextArrow = <button key={1} ref='next' type="button" data-role="none" className={cx(nextClasses)} style={{display: 'block'}} onClick={nextHandler}>Next</button>;
+      var prevArrow = <button key={0} ref='previous' type="button" data-role="none" className={classnames(prevClasses)} style={{display: 'block'}} onClick={prevHandler}> Previous</button>;
+      var nextArrow = <button key={1} ref='next' type="button" data-role="none" className={classnames(nextClasses)} style={{display: 'block'}} onClick={nextHandler}>Next</button>;
       return [prevArrow, nextArrow];
@@ -142,4 +142,5 @@ } else {
   render: function () {
+    var className = classnames('slick-initialized', 'slick-slider', this.props.className);
     return (
-      <div className={'slick-initialized slick-slider ' + this.props.className} >
+      <div className={className} >
         <div
@@ -146,0 +147,0 @@ ref='list'

lib/mixins/helpers.js

 var assign = require('object-assign');
 var React = require('react');
-var cx = require('react/lib/cx');
+var classnames = require('classnames');
 var ReactTransitionEvents = require('react/lib/ReactTransitionEvents');
@@ -106,2 +106,3 @@
       transform: 'translate3d(' + targetLeft + 'px, 0px, 0px)',
+      transition: ''
     };
@@ -144,3 +145,3 @@
     }
-    return cx({
+    return classnames({
       'slick-slide': true,
@@ -208,2 +209,10 @@ 'slick-active': slickActive,
 
+    var callback = function() {  
+        this.setState({
+          animating: false,
+          trackStyle: this.getCSS(currentLeft),
+          swipeLeft: null
+        });
+    }.bind(this)
+
     this.setState({
@@ -215,9 +224,4 @@ animating: true,
     }, function () {
-      ReactTransitionEvents.addEndEventListener(this.refs.track.getDOMNode(), function() {  
-        this.setState({
-          animating: false,
-          trackStyle: this.getCSS(currentLeft),
-          swipeLeft: null
-        });
-      }.bind(this));
+      ReactTransitionEvents.addEndEventListener(this.refs.track.getDOMNode(), callback);
+      setTimeout(callback, this.props.speed);
     });
@@ -224,0 +228,0 @@

package.json

 {
   "name": "react-slick",
-  "version": "0.3.8",
+  "version": "0.3.9",
   "description": " React port of slick carousel",
@@ -55,2 +55,3 @@ "main": "index.jsx",
   "dependencies": {
+    "classnames": "^1.2.0",
     "json2mq": "^0.2.0",
@@ -57,0 +58,0 @@ "lodash.assign": "^3.0.0",

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Unidentified License

License

(Experimental) Something that seems like a license was found, but its contents could not be matched with a known license.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Unidentified License

License

(Experimental) Something that seems like a license was found, but its contents could not be matched with a known license.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

8896292

Lines of code

151380

Worsened metrics

Dependency count

10

increased by11.11%

Dependency changes

• + Addedclassnames@^1.2.0

• + Addedclassnames@1.2.2(transitive)