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

feat: add props from native Screen component #9458

Closed
wants to merge 1,759 commits into from

Conversation

WoLewicki
Copy link
Member

@WoLewicki WoLewicki commented Mar 25, 2021

PR adding options exposed to the Screen component of react-native-screens in software-mansion/react-native-screens#863. It gives the user the ability to control orientation and status bar management per screen of the navigator.

Changes:

  • Added NativeScreenTraitsProps type to stack&drawer&tabs navigators
  • Bump example and stack&drawer&tabs to use react-native-screens v3.3.0
  • Hardcoded types from RNS library directly to native-stack to avoid cross-versioning issues
  • Switched from using react-native's StatusBar to react-native-screens' in example app

@netlify
Copy link

netlify bot commented Mar 25, 2021

✔️ Deploy Preview for react-navigation-example ready!

🔨 Explore the source changes: 2da692d

🔍 Inspect the deploy log: https://app.netlify.com/sites/react-navigation-example/deploys/61f967f24b61bd000823e228

😎 Browse the preview: https://deploy-preview-9458--react-navigation-example.netlify.app/

@github-actions
Copy link

The Expo app for the example from this branch is ready!

expo.io/@react-navigation/react-navigation-example?release-channel=pr-9458

satya164 and others added 27 commits May 9, 2021 04:56
 - @react-navigation/bottom-tabs@6.0.0-next.7
 - @react-navigation/core@6.0.0-next.4
 - @react-navigation/devtools@6.0.0-next.4
 - @react-navigation/drawer@6.0.0-next.6
 - @react-navigation/elements@1.0.0-next.6
 - @react-navigation/material-bottom-tabs@6.0.0-next.4
 - @react-navigation/material-top-tabs@6.0.0-next.5
 - @react-navigation/native@6.0.0-next.4
 - @react-navigation/stack@6.0.0-next.11
 - @react-navigation/bottom-tabs@6.0.0-next.8
 - @react-navigation/core@6.0.0-next.5
 - @react-navigation/devtools@6.0.0-next.5
 - @react-navigation/drawer@6.0.0-next.7
 - @react-navigation/elements@1.0.0-next.7
 - @react-navigation/material-bottom-tabs@6.0.0-next.5
 - @react-navigation/material-top-tabs@6.0.0-next.6
 - @react-navigation/native@6.0.0-next.5
 - @react-navigation/stack@6.0.0-next.12
 - @react-navigation/bottom-tabs@6.0.0-next.9
 - @react-navigation/core@6.0.0-next.6
 - @react-navigation/devtools@6.0.0-next.6
 - @react-navigation/drawer@6.0.0-next.8
 - @react-navigation/elements@1.0.0-next.8
 - @react-navigation/material-bottom-tabs@6.0.0-next.6
 - @react-navigation/material-top-tabs@6.0.0-next.7
 - @react-navigation/native@6.0.0-next.6
 - @react-navigation/stack@6.0.0-next.13
 - @react-navigation/stack@6.0.0-next.14
 - @react-navigation/bottom-tabs@6.0.0-next.10
 - @react-navigation/core@6.0.0-next.7
 - @react-navigation/devtools@6.0.0-next.7
 - @react-navigation/drawer@6.0.0-next.9
 - @react-navigation/elements@1.0.0-next.9
 - @react-navigation/material-bottom-tabs@6.0.0-next.7
 - @react-navigation/material-top-tabs@6.0.0-next.8
 - @react-navigation/native@6.0.0-next.7
 - @react-navigation/stack@6.0.0-next.15
satya164 and others added 24 commits January 28, 2022 16:25
Reanimated has a bug where it doesn't update the `transform` property from what was set before if the new style object doesn't include it. We don't want to add a `transform` when not needed due to browser bugs (https://bugs.chromium.org/p/chromium/issues/detail?id=20574). This commit adds `transform: undefined` explicitly to workaround the bug in Reanimated while still resulting on the style not being included to avoid the browser bug.

Fixes #10210
While passing a component with an uncapitalized name works, it can mean that the user is trying to pass a regular function and not a component. Regular functions are usually defined inside components causing state to be lost on re-render if used as components. This warning is supposed to catch those cases.
…10067)

Calling `removeListener` on event that doesn't have any listeners will add new key for that event to listeners with value `undefined`. After that attempts to set `ref.current` will fail with "Cannot read property 'forEach' of undefined" error. Fix is quite simple. Just needed to add check if there is listeners for the given event.

FYI @satya164. This is related to your recent change acdde18
 - @react-navigation/bottom-tabs@6.1.0
 - @react-navigation/core@6.1.1
 - @react-navigation/devtools@6.0.5
 - @react-navigation/drawer@6.2.0
 - @react-navigation/elements@1.3.0
 - flipper-plugin-react-navigation@1.3.8
 - @react-navigation/material-bottom-tabs@6.1.0
 - @react-navigation/material-top-tabs@6.1.0
 - @react-navigation/native-stack@6.3.0
 - @react-navigation/native@6.0.7
 - @react-navigation/stack@6.1.0
Add `drawerAllowFontScaling` to the drawer options, so consumers can enable or disable font scaling for drawer labels to respect Text Size accessibility settings. The option functions similarly to [`tabBarAllowFontScaling`](tabBarAllowFontScaling).

Co-authored-by: Kyle Edgette <kyle@join-real.com>
Added new prop `tabsOrDrawer` for `ScreenContainer` which makes `react-native-screens` on `iOS` use new component if it is available. See software-mansion/react-native-screens#1029 for context.
@satya164
Copy link
Member

Hello, the PR got closed automatically due to some changes in the main branch. Would you be able to send a new PR? Thank you.

@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 satya164 deleted the @wolewicki/add-new-screen-props branch March 12, 2024 23:45
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.

None yet