Skip to content

Commit 15d93c5

Browse files
authored
Merge pull request #2329 from rjsdnql123/fix/element-slidesToShow-same-value
Fix: If the element (DOM element) and the value of 'slidesToShow' are…
2 parents d91bb58 + 3669b0a commit 15d93c5

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

src/slider.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -198,7 +198,7 @@ export default class Slider extends React.Component {
198198
if (settings === "unslick") {
199199
const className = "regular slider " + (this.props.className || "");
200200
return <div className={className}>{children}</div>;
201-
} else if (newChildren.length <= settings.slidesToShow) {
201+
} else if (newChildren.length <= settings.slidesToShow && !settings.infinite) {
202202
settings.unslick = true;
203203
}
204204
return (

src/track.js

+1-4
Original file line numberDiff line numberDiff line change
@@ -138,8 +138,7 @@ const renderSlides = spec => {
138138
if (spec.infinite && spec.fade === false) {
139139
let preCloneNo = childrenCount - index;
140140
if (
141-
preCloneNo <= getPreClones(spec) &&
142-
childrenCount !== spec.slidesToShow
141+
preCloneNo <= getPreClones(spec)
143142
) {
144143
key = -preCloneNo;
145144
if (key >= startIndex) {
@@ -164,7 +163,6 @@ const renderSlides = spec => {
164163
);
165164
}
166165

167-
if (childrenCount !== spec.slidesToShow) {
168166
key = childrenCount + index;
169167
if (key < endIndex) {
170168
child = elem;
@@ -186,7 +184,6 @@ const renderSlides = spec => {
186184
}
187185
})
188186
);
189-
}
190187
}
191188
});
192189

0 commit comments

Comments
 (0)