Skip to content

Fix typos in modal.md, timers.md, colors.md#5089

Merged
Simek merged 2 commits into
facebook:mainfrom
qa2me:fix/typos-modal-backtick
May 18, 2026
Merged

Fix typos in modal.md, timers.md, colors.md#5089
Simek merged 2 commits into
facebook:mainfrom
qa2me:fix/typos-modal-backtick

Conversation

@qa2me
Copy link
Copy Markdown
Contributor

@qa2me qa2me commented May 18, 2026

Summary

Fixes three documentation typos:

  • modal.md: Missing closing backtick on \pageSheet` or `formSheet`
  • timers.md: Wrong article — "a iPhone" → "an iPhone"
  • colors.md: Non-idiomatic phrase — "same like" → "same as"

All changes are minor grammar/markdown fixes only.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

@qa2me is attempting to deploy a commit to the Meta Open Source Team on Vercel.

A member of the Team first needs to authorize it.

@meta-cla
Copy link
Copy Markdown

meta-cla Bot commented May 18, 2026

Hi @qa2me!

Thank you for your pull request and welcome to our community.

Action Required

In order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you.

Process

In order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA.

Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with CLA signed. The tagging process may take up to 1 hour after signing. Please give it that time before contacting us about it.

If you have received this in error or have any questions, please contact us at cla@meta.com. Thanks!

@netlify
Copy link
Copy Markdown

netlify Bot commented May 18, 2026

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 4645d81
🔍 Latest deploy log https://app.netlify.com/projects/react-native/deploys/6a0affd669b2c60007734f30
😎 Deploy Preview https://deploy-preview-5089--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
Copy link
Copy Markdown
Collaborator

Simek commented May 18, 2026

Hey @qa2me, thanks for the corrections! 👍

Can you backport he changes to versioned docs, so it will apply to all docs versions?

Also, to accept you contribution we require signing the CLA, check out bot comment for the details.

@meta-cla meta-cla Bot added the CLA Signed label May 18, 2026
@qa2me
Copy link
Copy Markdown
Contributor Author

qa2me commented May 18, 2026

Done — backported to all 10 versioned docs directories (version-0.77 through version-0.86). Same three fixes applied consistently across all versions.

Copy link
Copy Markdown
Collaborator

@Simek Simek left a comment

Choose a reason for hiding this comment

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

Thanks for the follow up, LGTM! 🚀

@Simek Simek merged commit 1f960bc into facebook:main May 18, 2026
6 of 7 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