Skip to content

Commit d17c1f1

Browse files
authored
Merge pull request #1857 from akiran/swipe-right-issue
fix an issue with slick-current when swiped from left to right on fir…
2 parents 2b44d88 + bef98be commit d17c1f1

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "react-slick",
3-
"version": "0.27.2",
3+
"version": "0.27.3",
44
"description": " React port of slick carousel",
55
"main": "./lib",
66
"files": [

src/utils/innerSliderUtils.js

+8-3
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,9 @@
11
import React from "react";
22

3+
export function clamp(number, lowerBound, upperBound) {
4+
return Math.max(lowerBound, Math.min(number, upperBound));
5+
}
6+
37
export const getOnDemandLazySlides = spec => {
48
let onDemandSlides = [];
59
let startIndex = lazyStartIndex(spec);
@@ -172,6 +176,7 @@ export const slideHandler = spec => {
172176
finalLeft;
173177
let state = {},
174178
nextState = {};
179+
const targetSlide = infinite ? index : clamp(index, 0, slideCount);
175180
if (fade) {
176181
if (!infinite && (index < 0 || index >= slideCount)) return {};
177182
if (index < 0) {
@@ -220,22 +225,22 @@ export const slideHandler = spec => {
220225
currentSlide: finalSlide,
221226
trackStyle: getTrackCSS({ ...spec, left: finalLeft }),
222227
lazyLoadedList,
223-
targetSlide: index
228+
targetSlide
224229
};
225230
} else {
226231
state = {
227232
animating: true,
228233
currentSlide: finalSlide,
229234
trackStyle: getTrackAnimateCSS({ ...spec, left: animationLeft }),
230235
lazyLoadedList,
231-
targetSlide: index
236+
targetSlide
232237
};
233238
nextState = {
234239
animating: false,
235240
currentSlide: finalSlide,
236241
trackStyle: getTrackCSS({ ...spec, left: finalLeft }),
237242
swipeLeft: null,
238-
targetSlide: index
243+
targetSlide
239244
};
240245
}
241246
}

0 commit comments

Comments
 (0)