Issue 189. next/prev buttons are now disabled if there is nothing to scroll

Author: tipiirai

src/scrollable/scrollable.js

@@ -125,6 +125,8 @@
 
 				if (!conf.circular)  {
 					itemWrap.append(item);
+					next.removeClass("disabled");
+					
 				} else {
 					itemWrap.children("." + conf.clonedClass + ":last").before(item);
 					itemWrap.children("." + conf.clonedClass + ":first").replaceWith(item.clone().addClass(conf.clonedClass)); 						
@@ -229,10 +231,9 @@
 
 		// next/prev buttons
 		var prev = find(root, conf.prev).click(function() { self.prev(); }),
-			 next = find(root, conf.next).click(function() { self.next(); });	
+			 next = find(root, conf.next).click(function() { self.next(); }); 
 
-		if (!conf.circular && self.getSize() > 1) {
-			
+		if (!conf.circular) {
 			self.onBeforeSeek(function(e, i) {
 				setTimeout(function() {
 					if (!e.isDefaultPrevented()) {
@@ -241,10 +242,14 @@
 					}
 				}, 1);
 			}); 
+		}
+		
+		if (!conf.initialIndex) {
+			prev.addClass(conf.disabledClass);	
+		}
 
-			if (!conf.initialIndex) {
-				prev.addClass(conf.disabledClass);	
-			}
+		if (self.getSize() < 2) {
+			prev.add(next).addClass(conf.disabledClass);	
 		}
 
 		// mousewheel support

test/scrollable/single.html

@@ -3,14 +3,21 @@
 <script src="../../src/scrollable/scrollable.js"></script>
 
 <link rel="stylesheet" type="text/css" href="style.css"/>
+<style>
+.disabled {
+	visibility:hidden;		
+}
+</style>
 
 <script>
 $(function() {
-	$(".scrollable").scrollable({ 
+	var api = $(".scrollable").scrollable({ 
 		onSeek: function() {
 			console.info("jou");
 		}
-	}); 	
+	}).data("scrollable"); 	
+	
+	api.addItem("<div>added</div>");
 });
 </script>
 
@@ -25,12 +32,12 @@
 
 		<div class="items">
 			<div>0</div>  
-			<div>1</div>  
+			<!-- <div>1</div>  
 			<div>2</div>  
 			<div>3</div>  
 			<div>4</div>
 			<div>5</div>  
-			<div>6</div>  
+			<div>6</div>   -->
 		</div>
 
 	</div>