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.

Sign up for GitHub

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

Jump to bottom

Swipe back not triggering ionViewWillEnter #8958

Closed
neelz040 opened this issue Oct 30, 2016 · 1 comment · Fixed by #8951
Closed

Swipe back not triggering ionViewWillEnter #8958

neelz040 opened this issue Oct 30, 2016 · 1 comment · Fixed by #8951
Assignees
@manucorporat
Milestone
2.0.0-rc.2

Comments

@neelz040
Copy link

Short description of the problem:

on a swipe back (iOS) from a view 'B' to its previous view 'A', the ionViewWillEnter is not triggered. This causes my refresher and searchbar on the view 'A' not to work. However, when going back from B to A with the back button on the top left there is no problem.

NOTE: ionViewDidEnter does trigger on both, swipe back and back button.

Anyone else having this issue or knows a solution? Thanks

Cordova CLI: 6.3.1
Ionic Framework Version: 2.0.0-rc.1
Ionic CLI Version: 2.1.0
Ionic App Lib Version: 2.1.0-beta.1
ios-deploy version: 1.9.0 
ios-sim version: 5.0.8 
OS: Mac OS X El Capitan
Node Version: v6.8.1
Xcode version: Xcode 8.1 Build version 8B62 

WKWebView
@manucorporat manucorporat self-assigned this Oct 30, 2016
@manucorporat manucorporat added this to the 2.0.0-rc.2 milestone Oct 30, 2016
manucorporat added a commit to manucorporat/ionic that referenced this issue Oct 31, 2016
@manucorporat
fix(nav): swipe to go back gesture
415f607 
- 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
manucorporat added a commit to manucorporat/ionic that referenced this issue Oct 31, 2016
@manucorporat
fix(nav): swipe to go back gesture
cfb2fc5 
- 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
manucorporat added a commit to manucorporat/ionic that referenced this issue Oct 31, 2016
@manucorporat
fix(nav): swipe to go back gesture
a0f4d19 
- 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 mentioned this issue Oct 31, 2016
Merged
manucorporat added a commit to manucorporat/ionic that referenced this issue Oct 31, 2016
@manucorporat
fix(nav): swipe to go back gesture
f65d838 
- 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 added a commit to manucorporat/ionic that referenced this issue Nov 1, 2016
@manucorporat
fix(nav): swipe to go back gesture
1bbcc4f 
- 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
@netcell netcell mentioned this issue Nov 1, 2016
Closed
manucorporat added a commit to manucorporat/ionic that referenced this issue Nov 1, 2016
@manucorporat
fix(nav): swipe to go back gesture
a518049 
- 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 added a commit to manucorporat/ionic that referenced this issue Nov 1, 2016
@manucorporat
fix(nav): swipe to go back gesture
ad24b79 
- 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 added a commit to manucorporat/ionic that referenced this issue Nov 1, 2016
@manucorporat
fix(nav): swipe to go back gesture
02d5a28 
- 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 added a commit to manucorporat/ionic that referenced this issue Nov 1, 2016
@manucorporat
fix(nav): swipe to go back gesture
1af6de1 
- 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
adamdbradley pushed a commit that referenced this issue Nov 1, 2016
@manucorporat @adamdbradley
fix(nav): swipe to go back gesture
04d61ee 
- 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 #8919
fixes #8958
fixes #7934
@markisme
Copy link

Does that was fixed?I got the same problem. When i swipe to go back, the refresher doesn't work, but back from the back button work fine.

@ionitron-bot ionitron-bot bot locked and limited conversation to collaborators Sep 9, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@manucorporat manucorporat

Labels
None yet
Projects
None yet
Milestone
2.0.0-rc.2
Development

Successfully merging a pull request may close this issue.

fix(nav): swipe to go back gesture manucorporat/ionic
3 participants
@manucorporat @markisme @neelz040