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

using swipe to switch pages#92

Open
floridemai wants to merge 2 commits into
masterfrom
issue-90-swipe
Open

using swipe to switch pages#92
floridemai wants to merge 2 commits into
masterfrom
issue-90-swipe

Conversation

@floridemai
Copy link
Copy Markdown

@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
Copy Markdown

@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
Copy Markdown
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