Skip to content

Update memoryWarning event description to mention it is iOS only#4972

Merged
Simek merged 3 commits into
react:mainfrom
chetstone:patch-1
Jan 24, 2026
Merged

Update memoryWarning event description to mention it is iOS only#4972
Simek merged 3 commits into
react:mainfrom
chetstone:patch-1

Conversation

@chetstone

Copy link
Copy Markdown
Contributor

This event is iOS only, and the current documentation does not mention that fact. Also, it incorrectly states that the event fires when when a memory warning is "thrown or released". In fact, there is no event when memory pressure is released.

See react/react-native#36426 and react/react-native#47951

This event is iOS only, and the current documentation does not mention that fact. Also, it incorrectly states that the event fires when when a memory warning is "thrown or released". In fact, there is no event when memory pressure is released.

See react/react-native#36426 and react/react-native#47951
@meta-cla meta-cla Bot added the CLA Signed label Jan 22, 2026
@netlify

netlify Bot commented Jan 22, 2026

Copy link
Copy Markdown

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 13d9b05
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/69750488aa4357000829580a
😎 Deploy Preview https://deploy-preview-4972--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.

@Simek Simek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Hey @chetstone, thanks for making this correction! 👍

Can you also update the corresponding files in versioned docs, located in:

Apply the same changes to all versioned documentation (0.77-0.83):
- Add iOS platform label to memoryWarning event
- Improve event description clarity
@Simek

Simek commented Jan 23, 2026

Copy link
Copy Markdown
Collaborator

Thanks for the quick follow up! Can you fix the lint issues and with that addressed PR should be ready to merge.

Comment thread docs/appstate.md Outdated
Comment thread website/versioned_docs/version-0.77/appstate.md Outdated
Comment thread website/versioned_docs/version-0.78/appstate.md Outdated
Comment thread website/versioned_docs/version-0.79/appstate.md Outdated
Comment thread website/versioned_docs/version-0.80/appstate.md Outdated
Comment thread website/versioned_docs/version-0.81/appstate.md Outdated
Comment thread website/versioned_docs/version-0.82/appstate.md Outdated
Comment thread website/versioned_docs/version-0.83/appstate.md Outdated
@Simek Simek merged commit 449d731 into react:main Jan 24, 2026
5 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.

2 participants