From 7b495cccf136f01f522d4bcf12122c8cc4a01675 Mon Sep 17 00:00:00 2001
From: Antonio Laguna <sombragriselros@gmail.com>
Date: Sat, 28 Jan 2017 15:38:21 +0100
Subject: [PATCH] Navigation to plugins

---
 src/modules/navigation.js | 24 +++++++++++++++++---
 src/modules/plugins.js    |  5 +++++
 src/modules/webslides.js  | 47 ++++++++++++++++++++++++++-------------
 3 files changed, 57 insertions(+), 19 deletions(-)
 create mode 100644 src/modules/plugins.js

diff --git a/src/modules/navigation.js b/src/modules/navigation.js
index 19d04de..1ce624d 100644
--- a/src/modules/navigation.js
+++ b/src/modules/navigation.js
@@ -18,10 +18,15 @@ const LABELS = {
   }
 };
 
-
 export default class Navigation {
-  constructor({ isVertical }) {
-    const arrowLabels = isVertical ? LABELS.VERTICAL : LABELS.HORIZONTAL;
+  /**
+   * The Navigation constructor. It'll create all the nodes needed for the
+   * navigation such as the arrows and the counter.
+   * @param {WebSlides} wsInstance The WebSlides instance
+   */
+  constructor(wsInstance) {
+    const arrowLabels = wsInstance.isVertical ?
+        LABELS.VERTICAL : LABELS.HORIZONTAL;
 
     this.el = DOM.createNode('div', 'navigation');
     this.next = Navigation.createArrow(ELEMENT_ID.NEXT, arrowLabels.NEXT);
@@ -31,12 +36,25 @@ export default class Navigation {
     this.el.appendChild(this.next);
     this.el.appendChild(this.prev);
     this.el.appendChild(this.counter);
+
+    wsInstance.el.appendChild(this.el);
   }
 
+  /**
+   *
+   * @param current
+   * @param max
+   */
   updateCounter(current, max) {
     this.counter.textContent = `${current} / ${max}`;
   }
 
+  /**
+   * Creates an arrow to navigate.
+   * @param {!String} id Desired ID for the arrow.
+   * @param {!String} text Desired text for the arrow.
+   * @return {Element} The arrow element.
+   */
   static createArrow(id, text) {
     const arrow = DOM.createNode('a', id, text);
     arrow.href = '#';
diff --git a/src/modules/plugins.js b/src/modules/plugins.js
new file mode 100644
index 0000000..4b0afd9
--- /dev/null
+++ b/src/modules/plugins.js
@@ -0,0 +1,5 @@
+import Navigation from './navigation';
+
+export default {
+  Navigation
+};
diff --git a/src/modules/webslides.js b/src/modules/webslides.js
index d001481..7d7bcf6 100644
--- a/src/modules/webslides.js
+++ b/src/modules/webslides.js
@@ -1,5 +1,5 @@
 import Hash from './hash';
-import Navigation from './navigation';
+import Plugins from './plugins';
 import Slide from './slide';
 import DOM from '../utils/dom';
 import ScrollHelper from '../utils/scroll-to';
@@ -8,16 +8,18 @@ const CLASSES = {
   VERTICAL: 'vertical'
 };
 
+const PLUGINS = ['Navigation'];
+
 export default class WebSlides {
   constructor() {
     this.el = document.getElementById('webslides');
     this.isMoving = false;
     this.slides = null;
-    this.navigation = null;
     this.currentSlideI_ = -1;
     this.currentSlide_ = null;
     this.maxSlide_ = 0;
     this.isVertical = this.el.classList.contains(CLASSES.VERTICAL);
+    this.plugins = {};
 
     if (!this.el) {
       throw new Error('Couldn\'t find the webslides container!');
@@ -25,10 +27,10 @@ export default class WebSlides {
 
     this.removeChildren_();
     this.grabSlides_();
-    this.createNav_();
+    this.createPlugins_();
     this.initSlides_();
 
-    window.st = ScrollHelper;
+    Hash.init(this);
   }
 
   removeChildren_() {
@@ -44,12 +46,15 @@ export default class WebSlides {
     }
   }
 
-  createNav_() {
-    this.navigation = new Navigation({
-      isVertical: this.isVertical
+  createPlugins_() {
+    PLUGINS.forEach(pluginName => {
+      if (Plugins[pluginName]) {
+        const pluginCto = Plugins[pluginName];
+        this.plugins[pluginCto] = new pluginCto(this);
+      } else {
+        throw new Error(`Tried to initialize plugin ${pluginName} but doesn't exist.`);
+      }
     });
-
-    this.el.appendChild(this.navigation.el);
   }
 
   grabSlides_() {
@@ -67,11 +72,10 @@ export default class WebSlides {
       if (forward !== null) {
         isMovingForward = forward;
       } else {
-        if (Number.isInteger(this.currentSlideI_)) {
+        if (this.currentSlideI_ >= 0) {
           isMovingForward = slideI > this.currentSlideI_;
         }
       }
-
       const nextSlide = this.slides[slideI];
 
       if (this.currentSlide_ !== null) {
@@ -87,15 +91,19 @@ export default class WebSlides {
   animateToSlide_(isMovingForward, nextSlide, callback) {
     DOM.lockScroll();
 
-    nextSlide.show();
+    if (!isMovingForward) {
+      nextSlide.moveBeforeFirst();
+      nextSlide.show();
+      ScrollHelper.scrollTo(this.currentSlide_.el.offsetTop, 0);
+    } else {
+      nextSlide.show();
+    }
 
     ScrollHelper.scrollTo(nextSlide.el.offsetTop, 500, () => {
       this.currentSlide_.hide();
 
       if (isMovingForward) {
         this.currentSlide_.moveAfterLast();
-      } else {
-        nextSlide.moveBeforeFirst();
       }
 
       DOM.unlockScroll();
@@ -122,8 +130,6 @@ export default class WebSlides {
   onSlideChange_(slide) {
     this.currentSlide_ = slide;
     this.currentSlideI_ = slide.i;
-    this.navigation.updateCounter(
-      this.currentSlideI_ + 1, this.maxSlide_);
     this.isMoving = false;
 
     Hash.setSlideNumber(this.currentSlideI_ + 1);
@@ -162,6 +168,15 @@ export default class WebSlides {
       slideNumber = 0;
     }
 
+    // Keeping the order
+    if (slideNumber !== 0) {
+      let i = 0;
+      while(i < slideNumber) {
+        this.slides[i].moveAfterLast();
+        i++;
+      }
+    }
+
     this.goToSlide(slideNumber);
   }
 }