Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Difficult to swipe #19

Open
mstrelan opened this issue Mar 16, 2014 · 7 comments
Open

Difficult to swipe #19

mstrelan opened this issue Mar 16, 2014 · 7 comments
Labels
Milestone

Comments

@mstrelan
Copy link

Is it just me or is it really difficult to change slides? It seems you have to very particular in your swiping motion and timing. Immediately after completing a swipe you have to wait a second or so before going again, and you need to make sure you slide about 80% of the screen. I'm on a Nexus 5, anyone else experiencing this?

@apemsel
Copy link

apemsel commented Mar 17, 2014

Funny, on iOS it is almost too easy to swipe more than one image at a time ;-)

@ixisio
Copy link
Owner

ixisio commented Mar 17, 2014

@apemsel really, you could swipe multiple image with one gesture?
@mstrelan I got this behavior if I add to many images to the carousel, 10+. Can you try with only 3 to 5 images!? Is it the same here

@ixisio ixisio added the bug label Mar 17, 2014
@ixisio ixisio added this to the v1.0 milestone Mar 17, 2014
@mstrelan
Copy link
Author

Well actually I'm not using images, I'm using HTML content. In the end I used another solution because there were too many issues with this one (eg not able to click links in the slide content). Specifically (for anyone interested) I'm using JQuery TouchSwipe with this snippet.

    // Add touch events to the carousel.
    $('.carousel-inner').swipe({
      swipeLeft: function() {
        $(this).parent().carousel('next');
      },
      swipeRight: function() {
        $(this).parent().carousel('prev');
      },
      allowPageScroll: 'vertical',
      threshold: 25
    });

@apemsel
Copy link

apemsel commented Mar 17, 2014

@ixisio
I cannot confirm this with vanilla bootstrap-touch-carousel right now. Unfortunately we had quite some bugs to fix to make it work for our current project. I am trying to clean this up and create pull requests for some of the fixes (the first was the event triggering stuff).

@ixisio
Copy link
Owner

ixisio commented Jul 22, 2014

@apemsel talking about this PR: #18?

@apemsel
Copy link

apemsel commented Jul 22, 2014

yes

@ixisio
Copy link
Owner

ixisio commented Jul 25, 2014

@apemsel so it's not good to merge, aye?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants