Skip to content

Conversation

@kmichalikk
Copy link
Contributor

Docs update to match changes in react-native-screens software-mansion/react-native-screens#3189.

Motivation

In RN Screens, we fixed the bug with delay when dismissing full screen modals, and updated the docs for transitionDuration to reflect that we don't support customizing it for modals. This PR adds matching change to animationDuration doc.

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

Hey @kmichalikk! Thanks for opening your first pull request in this repo. If you haven't already, make sure to read our contribution guidelines.

@netlify
Copy link

netlify bot commented Sep 9, 2025

Deploy Preview for react-navigation-example ready!

Name Link
🔨 Latest commit 5634ed1
🔍 Latest deploy log https://app.netlify.com/projects/react-navigation-example/deploys/69033fee05c32e0008b3ed71
😎 Deploy Preview https://deploy-preview-12747--react-navigation-example.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Member

@satya164 satya164 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@satya164 satya164 force-pushed the @kmichalikk/animation-duration-transparent-modal-docs branch from 65bfdf2 to 5634ed1 Compare October 30, 2025 10:37
@satya164 satya164 enabled auto-merge (squash) October 30, 2025 10:37
@satya164 satya164 disabled auto-merge October 30, 2025 10:37
@satya164 satya164 merged commit f736427 into react-navigation:main Oct 30, 2025
9 of 11 checks passed
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.39%. Comparing base (10a5448) to head (5634ed1).
⚠️ Report is 31 commits behind head on main.
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12747      +/-   ##
==========================================
+ Coverage   74.13%   74.39%   +0.26%     
==========================================
  Files         223      225       +2     
  Lines        7295     7448     +153     
  Branches     2906     2992      +86     
==========================================
+ Hits         5408     5541     +133     
- Misses       1832     1853      +21     
+ Partials       55       54       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

Hey! This issue is closed and isn't watched by the core team. You are welcome to discuss the issue with others in this thread, but if you think this issue is still valid and needs to be tracked, please open a new issue with a repro.

satya164 added a commit that referenced this pull request Oct 30, 2025
Docs update to match changes in react-native-screens
software-mansion/react-native-screens#3189.

**Motivation**

In RN Screens, we fixed the bug with delay when dismissing full screen
modals, and updated the docs for `transitionDuration` to reflect that we
don't support customizing it for modals. This PR adds matching change to
`animationDuration` doc.

---------

Co-authored-by: Satyajit Sahoo <[email protected]>
satya164 added a commit that referenced this pull request Nov 1, 2025
Docs update to match changes in react-native-screens
software-mansion/react-native-screens#3189.

**Motivation**

In RN Screens, we fixed the bug with delay when dismissing full screen
modals, and updated the docs for `transitionDuration` to reflect that we
don't support customizing it for modals. This PR adds matching change to
`animationDuration` doc.

---------

Co-authored-by: Satyajit Sahoo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants