-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
bug: Back button missing sometimes - works with "ionic": "v1.0.0-rc.3" #4132
Comments
Just tried:
Which works, but:
doesn't :( |
I have same issue with version 1.0.1 Here is code pen demo. http://codepen.io/anon/pen/wamRVX4 Sequence to reproduce
|
Plunker Ionic 1.0.1: http://plnkr.co/edit/wzh9enXseYHfhakdxXhm?p=preview |
Plunker "Playlists - Ionic v1.0.0": |
@Robinyo so whats the difference between these two plunkrs? This one, http://plnkr.co/edit/pGwucVrTBDxSvw6z092h?p=preview , works as expected. |
That's correct, Plunker "Playlists - Ionic v1.0.0" - http://plnkr.co/edit/pGwucVrTBDxSvw6z092h?p=preview displays a '<' back button if you:
If you follow the same steps with the Ionic v1.0.1 Plunker - http://plnkr.co/edit/wzh9enXseYHfhakdxXhm?p=preview it displays a 'nav-icon' NOT a '<' back button. So, the 'bug' was introduced in Ionic version 1.0.1. |
There's a small list of changes in 1.0.1, and only two that may be the culprit. https://github.com/driftyco/ionic/blob/master/CHANGELOG.md#101-vanadium-vaquita-2015-06-30 |
Okay, so cef6708 is the culprit, looking into a fix |
@perrygovier I am still seeing this behavior in ionic v1.1.0 where eventually I will see this in the console: TypeError: Cannot read property 'expire' of null at ionic.bundle.js:47159 at Scope.$broadcast (ionic.bundle.js:24992) at y.transition.M.then.y.transition.y.transition (angular-ui-router.min.js:7) at processQueue (ionic.bundle.js:23394) at ionic.bundle.js:23410 at Scope.$eval (ionic.bundle.js:24673) at Scope.$digest (ionic.bundle.js:24484) at Scope.$apply (ionic.bundle.js:24778) at ionic.bundle.js:26545 at completeOutstandingRequest (ionic.bundle.js:14221) When using the menu-toggle attribute it is working fine. |
This issue remains unresolved. Using Ionic 1.7.12. The error occurs in nextViewOption() in the deregisterStateChangeListener. When a $stateChangeSuccess event occurs it's handler is called. nextViewOptions is null when the event occurs which triggers the error at nextViewOptions.expire when calling $timeout().
|
This occurs to me to, on Ionic 1.7.14. |
Me too.. v 1.7.14
|
Type: bug
Platform: all
See -> http://forum.ionicframework.com/t/back-button-missing-sometimes/28235/9
@yewnork - Its a BUG 😦
@cormacodonnell
@DonEllingsworth
@harmonics
See this fork of your Plunker: http://plnkr.co/edit/JVvifeOX5VSDGoa8lrTJ?p=preview
I updated the Plunker to use 1-0-1 (http://forum.ionicframework.com/t/1-0-1-vanadium-vaquita-released/27420) which didn't work so I checked my bower.json and noticed that I am still using:
which works, index.html:
Note: I also updated from href to ui-sref in several files + fixed 'menu.html' as well :)
The text was updated successfully, but these errors were encountered: