Offset calculation when slideCount == slidesToShow

Hi Ken,

First of all thanks for this superb plugin, really appreciated!

I ran into problems with v1.3.15 (v1.4 and v1.4.1 don’t seem to fix it either), when my SlideCount was equal to SlidesToShow (and CenterMode = true and Infinite = true). In this case it was showing only 4 out of 5 slides because on the left a extra margin was added with translate3D.

When looking through your code i found the problem in the getLeft() function.

if (_.slideCount <= _.options.slidesToShow){
            _.slideOffset = 0;
            verticalOffset = 0;
}

Ok, so now the slideOffset = 0;

if (_.options.centerMode === true && _.options.infinite === true) {
            _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2) - _.slideWidth;
        } else if (_.options.centerMode === true) {
            _.slideOffset = 0;
            _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2);
}

So now the slider gets added 1 slideWidth as Offset on the left side, ergo: showing 4 of 5 slides (last one is “hidden”), while this should be no Offset.

Is this something you can fix for upcomming versions?

[danodm]

I’ve managed to modify the getLeft() function so that it calculates and centres correctly based on whether slideCount is less than or equal to slidesToShow.

on line 1037 I modified the if statement from

if (_.options.centerMode === true && _.options.infinite === true) {
     _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2) - _.slideWidth;
} else if (_.options.centerMode === true) {
    _.slideOffset = 0;
    _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2);
}

to

if (_.options.centerMode === true && _.options.infinite === true) {
    if (_.slideCount <= _.options.slidesToShow) {
        _.slideOffset = ((_.slideWidth * Math.floor(_.options.slidesToShow)) / 2) - ((_.slideWidth * _.slideCount) / 2);
    } else {
         _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2) - _.slideWidth;
    }
} else if (_.options.centerMode === true) {
   _.slideOffset = 0;
   _.slideOffset += _.slideWidth * Math.floor(_.options.slidesToShow / 2);
}

Now the behaviour of the slide is as follows.

If slideCount is less than or equal to slideToShow centre the overall width of all items, else centre to the currently active slide.

[leggomuhgreggo]

Fixed in #2118.