fix(Slider): correct arrow disabled state in some case, close #1065

alibaba-fusion · Aug 29, 2019 · a9b319b · a9b319b
1 parent 00bbfe6
commit a9b319b
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 3 deletions.
diff --git a/src/slider/slick/arrow.jsx b/src/slider/slick/arrow.jsx
@@ -51,9 +51,9 @@ export default class Arrow extends Component {
         }
 
         // 下一个 index 大于总数？？
-        if (slideCount <= slidesToShow) {
-            return true;
-        }
+        // if (slideCount <= slidesToShow) {
+        //     return true;
+        // }
 
         // 向前箭头：当前是第 0 个
         if (type === 'prev') {

diff --git a/test/slider/index-spec.js b/test/slider/index-spec.js
@@ -361,6 +361,23 @@ describe('slider', function () {
             });
         });
 
+        it('should have correct disabled class for next/prev arrow', () => {
+            return co(function* () {
+                wrapper = mount(<Slider
+                    infinite={false}
+                    defaultActiveIndex={2}
+                    slidesToShow={5}
+                    >{slides}</Slider>);
+                yield delay(100);
+                assert(
+                    wrapper.find('.next-slick-arrow.next-slick-next').at(0).hasClass('disabled')
+                );
+                assert(
+                    !wrapper.find('.next-slick-arrow.next-slick-prev').at(0).hasClass('disabled')
+                );
+            });
+        });
+
         it('should hover next/prev arrow', () => {
             return co(function* () {
                 wrapper = mount(<Slider infinite={false}>{slides}</Slider>);