Fix: move previous and next check on Css, Html to JS for further feat…

…ures

bower.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-ui-carousel",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "authors": [
     {
       "name": "mihnsen",

dist/ui-carousel.css

@@ -88,8 +88,7 @@
   .ui-carousel .carousel-next .carousel-btn {
     right: -25px; }
   .ui-carousel .carousel-disable {
-    opacity: 0.5;
-    pointer-events: none; }
+    opacity: 0.5; }
     .ui-carousel .carousel-disable .carousel-btn:hover {
       opacity: .75; }
 

dist/ui-carousel.js

@@ -189,6 +189,10 @@ angular.module('ui.carousel.controllers').controller('CarouselController', ['$sc
    * - 8 total, 4 show, 3 scroll, current 1 => next index = 4
    */
   this.next = function () {
+    if (!_this.isClickableNext) {
+      return false;
+    }
+
     var indexOffset = _this.getIndexOffset();
     var slideOffset = indexOffset === 0 ? _this.options.slidesToScroll : indexOffset;
 
@@ -203,6 +207,10 @@ angular.module('ui.carousel.controllers').controller('CarouselController', ['$sc
    * @see next function
    */
   this.prev = function () {
+    if (!_this.isClickablePrev) {
+      return false;
+    }
+
     var indexOffset = _this.getIndexOffset();
     var slideOffset = indexOffset === 0 ? _this.options.slidesToScroll : _this.options.slidesToShow - indexOffset;
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "angular-ui-carousel",
-  "version": "0.1.5",
+  "version": "0.1.6",
   "author": {
     "name": "mihnsen",
     "email": "[email protected]"

src/demo/index.pug

@@ -26,7 +26,7 @@ html(ng-app="App")
       ga('create', 'UA-85040392-2', 'auto');
       ga('send', 'pageview');
 
-  body(ng-controller="CarouselDemoCtrl as ctrl")
+  body(ng-controller="CarouselDemoCtrl as DemoCtrl")
     header.header
       .container
         h1 UI-Carousel
@@ -53,18 +53,18 @@ html(ng-app="App")
           .divider
           .content-box
             h2.title Single Item
-            ui-carousel(slides="ctrl.single.slides", dots="true", on-init="ctrl.singleInit()", infinite="false")
+            ui-carousel(slides="DemoCtrl.single.slides", dots="true", on-init="DemoCtrl.singleInit()", infinite="false")
               carousel-item
                 h3 {{ item + 1 }}
             pre
               code.language-html(prism="")
-                | {{ ctrl.single.source }}
+                | {{ DemoCtrl.single.source }}
 
           .divider
           .content-box
             h2.title Multiple Item
             ui-carousel(
-              slides="ctrl.multiple.slides",
+              slides="DemoCtrl.multiple.slides",
               slides-to-show="3",
               slides-to-scroll="3",
               dots="true"
@@ -73,12 +73,12 @@ html(ng-app="App")
                 h3 {{ item + 1 }}
             pre
               code.language-html(prism="")
-                | {{ ctrl.multiple.source }}
+                | {{ DemoCtrl.multiple.source }}
           .divider
           .content-box
             h2.title Autoplay
             ui-carousel(
-              slides="ctrl.autoplay.slides",
+              slides="DemoCtrl.autoplay.slides",
               slides-to-show="3",
               slides-to-scroll="1",
               initial-slide="1",
@@ -90,12 +90,12 @@ html(ng-app="App")
                 h3 {{ item + 1 }}
             pre
               code.language-html(prism="")
-                | {{ ctrl.autoplay.source }}
+                | {{ DemoCtrl.autoplay.source }}
           .divider
           .content-box
             h2.title Fade
             ui-carousel(
-              slides="ctrl.fade.slides",
+              slides="DemoCtrl.fade.slides",
               slides-to-show="3",
               slides-to-scroll="2",
               fade="true",
@@ -106,13 +106,13 @@ html(ng-app="App")
                   img(src="{{ item }}")
             pre
               code.language-html(prism="")
-                | {{ ctrl.fade.source }}
+                | {{ DemoCtrl.fade.source }}
           .divider
           .content-box
             h2.title Customzie
             pre
               code.language-html(prism="")
-                | {{ ctrl.customize }}
+                | {{ DemoCtrl.customize }}
 
       section.screen.extra
         .divider

src/ui-carousel/controllers/carousel.controller.js

@@ -174,6 +174,10 @@ angular.module('ui.carousel.controllers')
      * - 8 total, 4 show, 3 scroll, current 1 => next index = 4
      */
     this.next = () => {
+      if (!this.isClickableNext) {
+        return false;
+      }
+
       const indexOffset = this.getIndexOffset();
       const slideOffset = indexOffset === 0
         ? this.options.slidesToScroll
@@ -192,6 +196,10 @@ angular.module('ui.carousel.controllers')
      * @see next function
      */
     this.prev = () => {
+      if (!this.isClickablePrev) {
+        return false;
+      }
+
       const indexOffset = this.getIndexOffset();
       const slideOffset = indexOffset === 0
         ? this.options.slidesToScroll

src/ui-carousel/scss/carousel.scss

@@ -50,7 +50,6 @@
   }
   .carousel-disable {
     opacity: 0.5;
-    pointer-events: none;
 
     .carousel-btn {
       &:hover {

test/unit/ui-carousel/controllers/carousel.controller.js

@@ -182,6 +182,7 @@ describe('ui.carousel.controller.CarouselController', function() {
         return $q.resolve(true);
       });
       CarouselCtrl.$onInit();
+      CarouselCtrl.isClickableNext = true;
     });
 
     it('should works well with normal case', () => {
@@ -228,6 +229,7 @@ describe('ui.carousel.controller.CarouselController', function() {
         return $q.resolve(true);
       });
       CarouselCtrl.$onInit();
+      CarouselCtrl.isClickablePrev = true;
     });
 
     it('should works well with infinite case and sync to page index first', () => {