nolimits4web / swiper

Most modern mobile touch slider with hardware accelerated transitions
https://swiperjs.com
MIT License
39.91k stars 9.75k forks source link

Bug gallery with Thumbs and Two way control when centeredslides: false #2483

Closed EdicionPG closed 4 years ago

EdicionPG commented 6 years ago

This is a (multiple allowed):

What you did

Using demo from gallery with Thumbs and 2 way control (http://idangero.us/swiper/demos/300-thumbs-gallery.html) selected centeredslides as false it doesnt work as expected

Expected Behavior

Normal function with thumbs aligned to left.

Actual Behavior

It fail when click on last thumbs.

the-fedorr commented 6 years ago

Having the same issue. It is totally broken when there is only few slides on the page and thumbs fits on a screen.

thomas-obernberger commented 6 years ago

same here!

purzlbaum commented 6 years ago

Same here.

remcotak commented 6 years ago

Same here! Would love to see a fix

kysonic commented 6 years ago

+1

perryutkonos commented 6 years ago

+1. solution - use Slick Slider

remcotak commented 6 years ago

Slick slider uses jQuery, that's why I wanted to use this library. Using Flickity now, does the job too.

https://flickity.metafizzy.co/

alesak commented 6 years ago

replaced native controller functionality with click handler on thumb slider:

on: {
    click: function (event) {
        if (galleryThumbs.clickedIndex !== undefined) {
            galleryTop.slideTo(galleryThumbs.clickedIndex);
        }
    }
}
morettolss commented 6 years ago

+1 for @alesak solution

stale[bot] commented 6 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

iztsv commented 6 years ago

any news about the issue?

remcotak commented 6 years ago

Try using Flickity, does the same thing and doesn't use jquery.

stale[bot] commented 5 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

thomas-obernberger commented 5 years ago

please fix the issue, dont want to move to an other library

ntucakovic commented 5 years ago

This is a big issue. We have to do a lot of workaround to achieve centeredSlides: false along with other behavior we might need to implement.

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] commented 4 years ago

This issue has been automatically closed due to inactivity. If this issue is still actual, please, create the new one.