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

Transition animation not working on android and WP #7171

Closed
xap5xap opened this issue Jul 2, 2016 · 4 comments
Closed

Transition animation not working on android and WP #7171

xap5xap opened this issue Jul 2, 2016 · 4 comments
Assignees
Milestone

Comments

@xap5xap
Copy link

xap5xap commented Jul 2, 2016

The animation "ios-transition" causes the returning page to occupy half of the screen when the back button is pressed. This only happens on Android and WP.

Steps to reproduce:
When I open another page with the code below, the animations works as expected from right to left on IOS, Android and WP, but when the back button is pressed on the opened page, then the parent’ content is rendered only the half.

        this.navController.push(SearchPage, null, { animate: true, animation: "ios-transition" });      

image

I am using Ionic 2.0.0-beta.10

Cordova CLI: 6.1.1
Gulp version: CLI version 3.9.1
Gulp local:
Ionic CLI Version: 2.0.0-beta.32
Ionic App Lib Version: 2.0.0-beta.18
OS:
Node Version: v4.4.6

@manucorporat
Copy link
Contributor

manucorporat commented Jul 3, 2016

Confirmed this is a issue in our e2e tests:
screen shot 2016-07-03 at 09 38 46

ios-transition leaves the ion-page's content behind in a bad state for the go back md-transition.

Possible solution:

  • "ios-transition" should reset content position after it finish
  • All transitions should reset page and content state before starting

@gadld
Copy link

gadld commented Aug 15, 2016

Hi, was this fixed? because I'm running beta.11 on ios 9.3.2 and still getting this issue

@schankam
Copy link

schankam commented Sep 8, 2016

Using beta11 and still getting the issue as well. Didn't see it happen on Android though, but on iOS.

@manucorporat
Copy link
Contributor

@schankam @gadld should be fixed thanks to: #8272

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

No branches or pull requests

5 participants