question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

bug: ion-nav transitions are broken in react

See original GitHub issue

Prerequisites

Ionic Framework Version

  • v4.x
  • v5.x
  • v6.x
  • Nightly

Current Behavior

Starting in 6.1.15, IonNav page transitions are incorrect.

The changes to support custom sized dialogs, introduced a change to the global stylesheet that is being applied in scenarios where IonNav is used within IonModal.

Problematic styles:

ion-modal .ion-page {
  position: relative;

  contain: layout style;

  height: 100%;
}

Expected Behavior

IonNav transitions should be consistent within a modal or outside of a modal.

Steps to Reproduce

  1. Clone: https://github.com/milang/ionic-simplevitejs
  2. Checkout branch: recursivemodalnav
  3. npm install
  4. npm run dev
  5. Click “Open Modal”
  6. Click “Push”
  7. Observe the incorrect transition between views

Code Reproduction URL

https://github.com/milang/ionic-simplevitejs

Ionic Info

6.1.15+

Additional Information

Related to: https://github.com/ionic-team/ionic-framework/issues/25677 (caused by the same change, but are effected differently between Angular and React).

Issue Analytics

  • State:closed
  • Created a year ago
  • Comments:5 (3 by maintainers)

github_iconTop GitHub Comments

1reaction
milangcommented, Jul 28, 2022

@liamdebeasi @sean-perkins I can confirm that development build 6.1.16-dev.11658962095.1f24647d works as expected in my environment. Thank you for the quick fix! I assume this fix should eventually be available in Ionic 6.2.1?

0reactions
ionitron-bot[bot]commented, Aug 27, 2022

Thanks for the issue! This issue is being locked to prevent comments that are not relevant to the original issue. If this is still an issue with the latest version of Ionic, please create a new issue and ensure the template is fully filled out.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Concurrent UI Patterns (Experimental) - React
Concurrent Mode offers a new set of tools to do that. Transitions. Wrapping setState in a Transition; Adding a Pending Indicator; Reviewing the...
Read more >
react-transition-group | Yarn - Package Manager
react -transition-group npm. ATTENTION! To address many issues that have come up over the years, the API in v2 and above is not...
Read more >
Can't resolve 'react-transition-group' in MUI IconButton
It looks like the react-transition-group 4.4.3 broke and may be missing project files. Downgrading to 4.4.2 worked for me.
Read more >
React Router V6 and Framer Motion Tutorial - YouTube
PAGE TRANSITION ANIMATIONS IN REACT ! In this video I will be showing how to use the framer motion library to create page...
Read more >
react-split-pane - npm
Split -Pane React component, can be nested or split vertically or horizontally! Installing. npm install react-split-pane.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found