Merge pull request #4 from dutchenkoOleg/master

3.0.0

Author: OlehDutchenko

index.js

@@ -55,7 +55,7 @@ export class AbstractSlickCarousel extends WebPluginInterface {
 	 */
 	get defaultProps () {
 		return {
-			pauseAutoPlayInOutOfView: true,
+			pauseAutoplayInOutOfView: false,
 			cssReadyClass: 'is-ready',
 			cssInitializedClass: 'is-initialized',
 			$listSelector: '[data-slick-carousel-list]',
@@ -126,18 +126,8 @@ export class AbstractSlickCarousel extends WebPluginInterface {
 	 * @protected
 	 */
 	_afterInitialize () {
-		if (this.props.pauseAutoPlayInOutOfView && this.settings.autoplay) {
-			this.$list.on('inview', (event, isInView) => {
-				if (this.isInitialized) {
-					if (isInView) {
-						this.update();
-						this.$list.slick('slickPlay');
-					} else {
-						this.$list.slick('slickPause');
-					}
-				}
-			});
-			this.$list.trigger('inview');
+		if (this.props.pauseAutoplayInOutOfView && this.settings.autoplay) {
+			this._autoplayInViewObserver();
 		}
 		this.$container.addClass(this.props.cssInitializedClass);
 	}
@@ -149,22 +139,43 @@ export class AbstractSlickCarousel extends WebPluginInterface {
 			if (this.firstInitialize) {
 				this._setup();
 				this.firstInitialize = false;
+				this._beforeInitialize();
 			}
-			this._beforeInitialize();
+
 			this.$list.slick(this.settings);
 			this.isInitialized = true;
-			this._afterInitialize();
+
+			if (this.firstInitialize) {
+				this._afterInitialize();
+			}
 		}
 	}
 
 	// ------------------------------
 	// extend interface
 	// ------------------------------
 
+	/**
+	 * @protected
+	 */
+	_autoplayInViewObserver () {
+		this.$list.on('inview', (event, isInView) => {
+			if (this.isInitialized) {
+				if (isInView) {
+					this.update();
+					this.$list.slick('slickPlay');
+				} else {
+					this.$list.slick('slickPause');
+				}
+			}
+		});
+		this.$list.trigger('inview');
+	}
+
 	destroy () {
 		if (this.isInitialized) {
 			this.$list.unslick();
-			this.$list.removeClass(this.props.cssReadyClass);
+			this.isInitialized = false;
 		}
 	}
 
@@ -181,7 +192,7 @@ export class AbstractSlickCarousel extends WebPluginInterface {
 
 /**
  * @typedef {Object} SlickCarouselProps
- * @property {boolean} [pauseAutoPlayInOutOfView] - true,
+ * @property {boolean} [pauseAutoplayInOutOfView] - true,
  * @property {string} [cssReadyClass] - 'is-ready',
  * @property {string} [cssInitializedClass] - 'is-initialized',
  * @property {string} [$listSelector] - '[data-slick-carousel-list]',

package.json

@@ -8,22 +8,22 @@
   },
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/dutchenkoOleg/web-plugin-abstract-slick-carousel.git"
+    "url": "git+https://github.com/WezomAgency/web-plugin-abstract-slick-carousel.git"
   },
   "keywords": [],
   "author": "Oleg Dutchenko <[email protected]>",
   "license": "MIT",
   "bugs": {
-    "url": "https://github.com/dutchenkoOleg/web-plugin-abstract-slick-carousel/issues"
+    "url": "https://github.com/WezomAgency/web-plugin-abstract-slick-carousel/issues"
   },
-  "homepage": "https://github.com/dutchenkoOleg/web-plugin-abstract-slick-carousel#readme",
+  "homepage": "https://github.com/WezomAgency/web-plugin-abstract-slick-carousel#readme",
   "devDependencies": {
     "eslint-config-wezom-relax": "^1.1.0"
   },
   "dependencies": {
     "jquery": "^3.3.1",
     "jquery-inview": "^1.1.2",
     "slick-carousel": "^1.8.1",
-    "web-plugin-interface": "^3.0.0"
+    "web-plugin-interface": "^3.0.1"
   }
 }