Skip to content
This repository has been archived by the owner on May 10, 2018. It is now read-only.

vertical slide method #378

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

vertical slide method #378

wants to merge 3 commits into from

Conversation

shenqihui
Copy link

use rn-carousel-vertical to choose vertical slide, default as Horizontal.

and the example is added.

thanks.

use  rn-carousel-vertical  to choose vertical slide, default as
Horizontal.
@willyboy
Copy link

I wrote something very similar to this last week. $swipe only triggers horizontal events so can I ask how you're handling this? Did you need to add another library?
I added the following: angular.element(iElement)
.on('mousedown touchstart',function(e){
coords = getEventCoords(e);
swipeStart(coords,e);
})
.on('mousemove touchmove',function(e){
coords = getEventCoords(e);
swipeMove(coords,e);
})
.on('mouseup touchend',function(e){
swipeEnd(coords,e);
});

Which required a few other changes.

@shenqihui
Copy link
Author

@willyboy sorry,at that time i won't note this .and i dont use that method.
And i looking into the code, i found that it use

                            $swipe.bind(iElement, {
                                start: swipeStart,
                                move: swipeMove,
                                end: swipeEnd,
                                cancel: function(event) {
                                    swipeEnd({}, event);
                                }
                            });

to detect the event.
but $swipe just detect left or right swipe, so we should choose another lib.

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

Successfully merging this pull request may close these issues.

2 participants