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

Update Performance Overview to Reflect High Refresh Rate Displays Issue #4062 #4259

Merged

Conversation

muhammadashfaq
Copy link
Contributor

PR Description:

What is addressed:

  • Updated the "Performance Overview > What you need to know about frames" section to reflect advancements in mobile device displays.
  • Previous phrasing emphasized a fixed 60fps target, which is outdated for modern devices offering 90Hz and 120Hz refresh rates.

How it is useful:

  • The updated guidance advises aiming for at least 60fps to ensure compatibility across both standard and high-refresh-rate displays.
  • Optimizing for higher refresh rates (90Hz, 120Hz) will enhance the user experience by providing smoother animations and more responsive interfaces.
  • This change ensures that developers can future-proof their apps for modern and upcoming devices, leading to better app performance across the board.

Closes issue #4062.

Additional improvements:

  • The phrasing has been refined to improve readability and provide more flexibility, making the section easier to understand for a broader audience.

Copy link

netlify bot commented Oct 9, 2024

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit a40e769
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/6707e659508cc50009dd09c8
😎 Deploy Preview https://deploy-preview-4259--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 site configuration.

@muhammadashfaq
Copy link
Contributor Author

@cortinico

docs/performance.md Outdated Show resolved Hide resolved
@cortinico cortinico merged commit dd0c155 into facebook:main Oct 10, 2024
3 checks passed
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