Skip to content

Capitalize Components in docs - #8666

Open
Dextheking1 wants to merge 4 commits into
reactjs:mainfrom
Dextheking1:docs/capitalize-components
Open

Dextheking1 wants to merge 4 commits into
reactjs:mainfrom
Dextheking1:docs/capitalize-components

Conversation

@Dextheking1

Copy link
Copy Markdown

Part of #6713 (Components item).

Capitalizes the React concept Component(s) where prose refers to the concept directly, applying @rickhanlonii's guidance from #6717 strictly (capitalize only where it "makes sense to link to the Component reference page"). Deliberately minimal: reviewed all ~3,600 occurrences and found the docs are already substantially capitalized for the clear concept uses, so this is 5 casing-only edits across 4 files:

  • learn/index.md: "React Components are JavaScript functions that return markup" (definitional)
  • learn/your-first-component.md: two "React Components are regular JavaScript functions" statements (matches the issue's "Functional Components are a core concept in React" example; consistent with the same file's already-capitalized line 175)
  • reference/react/Component.md and reference/react/PureComponent.md: "Class Components are still supported by React" (categorical statement about the concept's support status; matches the existing "Class Components:" labels in legacy.md)

Deliberately left lowercase after full review: descriptive/instance uses ("your/this/child/named components"), indefinite-article definitions ("A React component is..."), behavioral statements ("upgrade your apps one component at a time"), built-in browser components ("the component"), counts/idioms, "function/class components" in instructional phrasing (current docs convention), compound "subcomponent(s)", and everything in blog/, community/, and the homepage.

URLs, anchors, inline code, and identifiers untouched.

@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant