Update example and fix bug with goTo same index

peduarte · Sep 3, 2015 · b5ccfe1 · b5ccfe1
1 parent 481444e
commit b5ccfe1
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 6 deletions.
diff --git a/examples/animations.html b/examples/animations.html
@@ -17,11 +17,13 @@ <h2>Animations</h2>
 
     <!-- SLIDER STARTS -->
     <div class="Wallop Wallop--slide">
-      <div class="Wallop-item Wallop-item--current"><img src="imgs/1.jpg"></div>
-      <div class="Wallop-item"><img src="imgs/2.jpg"></div>
-      <div class="Wallop-item"><img src="imgs/3.jpg"></div>
-      <div class="Wallop-item"><img src="imgs/4.jpg"></div>
-      <div class="Wallop-item"><img src="imgs/5.jpg"></div>
+      <div class="Wallop-list">
+        <div class="Wallop-item Wallop-item--current"><img src="imgs/1.jpg"></div>
+        <div class="Wallop-item"><img src="imgs/2.jpg"></div>
+        <div class="Wallop-item"><img src="imgs/3.jpg"></div>
+        <div class="Wallop-item"><img src="imgs/4.jpg"></div>
+        <div class="Wallop-item"><img src="imgs/5.jpg"></div>
+      </div>
 
       <button class="Wallop-buttonPrevious">Previous</button>
       <button class="Wallop-buttonNext">Next</button>

diff --git a/js/Wallop.js b/js/Wallop.js
@@ -99,6 +99,8 @@
 
   // Method to add classes to the right elements depending on the index passed
   WS.goTo = function (index) {
+    if (index === this.currentItemIndex) { return; }
+
     // Check if it's a carousel and if so, change index to be last item when clicking previous on first item
     if (this.options.carousel && index === -1) { index = this.allItemsArrayLength - 1; }
     else if (index > this.allItemsArrayLength || index < 0) { return; }

diff --git a/js/Wallop.min.js b/js/Wallop.min.js