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

Add gesture navigation #82

Merged
merged 10 commits into from
Jun 26, 2023
Merged

Conversation

shocklateboy92
Copy link
Collaborator

Turns out there's a convenient library caled swipeable_page_route that adds exactly the behavior that other apps like Reddit Sync and Apollo had.

Disclaimer: I only tested it on the Linux build, as I can't build android packages on my machine due to some weird errors I don't have the energy to investigate right now.

Turns out there's a convenient library caled swipeable_page_route that
adds exactly the behavior that other aps like Reddit Sync and Apollo did.
@shocklateboy92
Copy link
Collaborator Author

Looks like someone created issue #59 requesting this functionality.

@Inovvia
Copy link

Inovvia commented Jun 26, 2023

Hi @shocklateboy92 I was able to build your branch and tested on android. Swiping does work on full posts.

I also couldn't fix the android errors when trying to build for my IDE, but I was able to create a release one using flutter build apk --flavor prod --target lib/main_prod.dart --release

@shocklateboy92 shocklateboy92 linked an issue Jun 26, 2023 that may be closed by this pull request
Copy link
Collaborator

@zachatrocity zachatrocity left a comment

Choose a reason for hiding this comment

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

LGTM

@zachatrocity zachatrocity merged commit b6b926b into liftoff-app:master Jun 26, 2023
5 checks passed
@TalOrenshtein
Copy link

How about making it optional? It conflicts with the device's gestures and make it go back twice

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

Successfully merging this pull request may close these issues.

Slide or swipe to dismiss/close a post
5 participants