ionic-team / ionic-framework

A powerful cross-platform UI toolkit for building native-quality iOS, Android, and Progressive Web Apps with HTML, CSS, and JavaScript.
https://ionicframework.com
MIT License
50.92k stars 13.52k forks source link

bug: Incorrect transition animation on 'forward' using Vue useIonRouter navigate/replace #24995

Open 3zzy opened 2 years ago

3zzy commented 2 years ago

Prerequisites

Ionic Framework Version

Current Behavior

In order to completely disable native back swipe in IOS I’m having to use replace instead of push and its working fine except for one issue … disorienting page transitions.

I expect IonRouter.navigate(url, "forward", "replace") to show a forward animation but instead it shows a back animation. Also the other issue I've noticed is that is that the very first transition almost always is abrupt ie. without any animation at all.

Here’s it in action: Screen Recording 2022 03 21 at 6 18 11 pm - YouTube 4

Expected Behavior

forward animation when using forward

Steps to Reproduce

Navigate between pages using IonRouter.navigate(url, "forward", "replace") in IonicVue

Code Reproduction URL

No response

Ionic Info

Ionic:

   Ionic CLI       : 6.19.0
   Ionic Framework : @ionic/vue 6.0.12

Capacitor:

   Capacitor CLI      : 3.4.3
   @capacitor/android : 3.4.3
   @capacitor/core    : 3.4.3
   @capacitor/ios     : 3.4.3

Utility:

   cordova-res : not installed globally
   native-run  : 1.5.0

System:

   NodeJS : v16.13.1 
   npm    : 8.4.0
   OS     : macOS Montere

Additional Information

No response

averyjohnston commented 2 years ago

Thanks for the issue. Could you provide a Github repo we can use to verify? This will also help us confirm the fix down the road.

ionitron-bot[bot] commented 2 years ago

Thanks for the issue! This issue has been labeled as needs reproduction. This label is added to issues that need a code reproduction.

Please reproduce this issue in an Ionic starter application and provide a way for us to access it (GitHub repo, StackBlitz, etc). Without a reliable code reproduction, it is unlikely we will be able to resolve the issue, leading to it being closed.

If you have already provided a code snippet and are seeing this message, it is likely that the code snippet was not enough for our team to reproduce the issue.

For a guide on how to create a good reproduction, see our Contributing Guide.

3zzy commented 2 years ago

@amandaesmith3 https://github.com/3zzy/test

averyjohnston commented 2 years ago

Thanks! I'm able to replicate both issues (no transition on the first navigation, then always using the back transition from then on).

3zzy commented 2 years ago

@amandaejohnston Wondering if there's any ETA for this bugfix? Thanks!

3zzy commented 2 years ago

I wonder how everyone else is able to workaround this major navigation issue. Is there a temporary solution while we wait for this to be fixed properly?

loremru commented 1 year ago

there isn't actually animation on replace