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

fix(nav): swipe to go back gesture #8951

Merged

Conversation

manucorporat
Copy link
Contributor

@manucorporat manucorporat commented Oct 29, 2016

DO NOT MERGE!

Changes proposed in this pull request:

  • smoother by debouncing touch events (reduces jank)
  • dynamic animation duration
  • intelligent behavior based in the position, speed and direccion of the swipe (sharing logic with sliding item)

Also, this PR features a very interesting performance improvement that soon could by applied to the sidemenu, reorder, sliding item... very promising!!

BEFORE:
screen shot 2016-10-31 at 20 23 52

AFTER:
screen shot 2016-10-31 at 20 21 11

fixes #8919
fixes #8958
fixes #7934

- smoother by debouncing touch events (reduces bank)
- dynamic animation duration
- intelligent behavior based in the position, speed and direccion of the swipe (sharing logic with sliding item)

fixes ionic-team#8919
fixes ionic-team#8958
fixes ionic-team#7934
@manucorporat manucorporat force-pushed the improved-go-back-swipe branch from 02d5a28 to 1af6de1 Compare November 1, 2016 18:22
@adamdbradley adamdbradley merged commit 04d61ee into ionic-team:master Nov 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants