Skip to content
This repository has been archived by the owner on Oct 5, 2022. It is now read-only.

using swipe to switch pages #92

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

using swipe to switch pages #92

wants to merge 2 commits into from

Conversation

floridemai
Copy link

@floridemai floridemai commented Feb 1, 2019

Issue: #90

I've added a new library (react-swipeable) that already has hooks support. I'll add another issue to update this once released.
It simply checks for swipe and redirects.
I've opened the PR to get early feedback, but there are two things i'm still looking at:

  • you can see that i've done a simple scrollTo(0,0) but i think this can be done better and also the testing for it is a workaround for now :/
    - you might have seen that i added a new page utils but i haven't refactored yet (in More) as the tests fail due to mocking not being done as it should. I'm looking into this atm.

@floridemai floridemai requested a review from hizo February 1, 2019 15:18
Copy link

@jh3y jh3y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not pulled it down to play with it but does the container transform as you swipe?

@floridemai
Copy link
Author

Not pulled it down to play with it but does the container transform as you swipe?

@jh3y Nopes, I just replicated what the "<" & ">" buttons do.

@jackdclark jackdclark self-requested a review February 4, 2019 11:14
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.

3 participants