From 932db62320c5fb4e692ec425cea781706999d31d Mon Sep 17 00:00:00 2001
From: Rijn <bxbian951122@gmail.com>
Date: Sat, 21 Jan 2017 20:40:31 -0600
Subject: [PATCH] remove autoplay direction prop fix clear interval bug

---
 src/components/carousel/carousel.vue | 17 +++++++----------
 test/routers/carousel.vue            |  9 ---------
 2 files changed, 7 insertions(+), 19 deletions(-)

diff --git a/src/components/carousel/carousel.vue b/src/components/carousel/carousel.vue
index 1f61de5..8dcc957 100644
--- a/src/components/carousel/carousel.vue
+++ b/src/components/carousel/carousel.vue
@@ -1,6 +1,6 @@
 <template>
     <div :class="classes">
-        <button :class="arrowClasses" class="left" @click="add(-1)">
+        <button :class="arrowClasses" class="left" @click="arrowEvent(-1)">
             <Icon type="chevron-left"></Icon>
         </button>
         <div :class="[prefixCls + '-list']">
@@ -8,7 +8,7 @@
                 <slot></slot>
             </div>
         </div>
-        <button :class="arrowClasses" class="right" @click="add(1)">
+        <button :class="arrowClasses" class="right" @click="arrowEvent(1)">
             <Icon type="chevron-right"></Icon>
         </button>
         <ul :class="dotsClasses">
@@ -47,13 +47,6 @@
                 type: Number,
                 default: 2000
             },
-            autoplayDirection: {
-                type: String,
-                default: 'left',
-                validator (value) {
-                    return oneOf(value, ['left', 'right']);
-                }
-            },
             easing: {
                 type: String,
                 default: 'ease'
@@ -197,6 +190,10 @@
                 index = index % this.slides.length;
                 this.currentIndex = index;
             },
+            arrowEvent (offset) {
+                this.setAutoplay();
+                this.add(offset);
+            },
             dotsEvent (event, n) {
                 if (event === this.trigger && this.currentIndex !== n) {
                     this.currentIndex = n;
@@ -208,7 +205,7 @@
                 window.clearInterval(this.timer);
                 if (this.autoplay) {
                     this.timer = window.setInterval(() => {
-                        this.add(this.autoplayDirection === 'left' ? 1 : -1);
+                        this.add(1);
                     }, this.autoplaySpeed);
                 }
             },
diff --git a/test/routers/carousel.vue b/test/routers/carousel.vue
index 12b8477..0171d80 100644
--- a/test/routers/carousel.vue
+++ b/test/routers/carousel.vue
@@ -12,13 +12,6 @@
             Speed <Slider :value.sync="autoplaySpeed" :min="300" :max="5000"></Slider>
         </i-col>
         <i-col span="4">
-            <p>Direction</p>
-            <Button-group>
-                <i-button @click="autoplayDirection = 'left'">Left</i-button>
-                <i-button @click="autoplayDirection = 'right'">Right</i-button>
-            </Button-group>
-        </i-col>
-        <i-col span="4">
             Switch To
             <Button-group>
                 <i-button @click="currentIndex = 0">0</i-button>
@@ -66,7 +59,6 @@
         :current-index.sync="currentIndex"
         :autoplay="autoplay"
         :autoplay-speed="autoplaySpeed"
-        :autoplay-direction="autoplayDirection"
         :dots="dots"
         :trigger="trigger"
         :arrow="arrow"
@@ -193,7 +185,6 @@
                 currentIndex: 0,
                 autoplay: true,
                 autoplaySpeed: 2000,
-                autoplayDirection: 'left',
                 remove: false,
                 pushItem: [],
                 arrow: 'hover',
--
libgit2 0.21.4