Skip to content

[docs] Performance: update outdated info #4591

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

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

jamonholmgren
Copy link
Contributor

The state of the Performance document is pretty outdated; this PR takes a step toward at least fixing or removing sections that would be misleading or confusing to React Native developers on the latest versions.

Changes

  • Changed very old Navigator / NavigatorIOS references to React Navigation
  • Changed class component references to function component
  • On lists, added references to FlashList/Legend List, which are broadly used in the community
  • Removed old-school optimizations for ListView
  • Removed references to enabling shouldRasterizeIOS and renderToHardwareTextureAndroid, as these are now enabled by default
  • Removed Slow navigator transitions section, which is about the removed Navigator animations.

Minor tweaks

  • Added links to relevant other documentation
  • Added iOS and Android where appropriate
  • Added alt text
  • Minor rewording

The state of the Performance document is pretty outdated; this PR takes a step toward at least fixing or removing sections that would be misleading or confusing to React Native developers on the latest versions.
Copy link

netlify bot commented Apr 23, 2025

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 7c872c1
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/682e4decd796930008d93342
😎 Deploy Preview https://deploy-preview-4591--react-native.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.

- Removes part about navigation transitions
- Removes reference to ListView
@jamonholmgren
Copy link
Contributor Author

Made those changes @satya164 ! Thank you for the review/suggestions.

Copy link
Contributor

@Abbondanzo Abbondanzo left a comment

Choose a reason for hiding this comment

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

One nit on a section being removed, otherwise looks great!


This is especially true when you have text with a transparent background positioned on top of an image, or any other situation where alpha compositing would be required to re-draw the view on each frame. You will find that enabling `shouldRasterizeIOS` or `renderToHardwareTextureAndroid` can help with this significantly.
Copy link
Contributor

Choose a reason for hiding this comment

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

The default value for renderToHardwareTextureAndroid is still false, I would keep this section in as it's particularly relevant when reaching for overlays like drop shadows or other render effects

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

Successfully merging this pull request may close these issues.

4 participants