Skip to content

Fix stylesheet precedence example - #7235

Merged
gaearon merged 2 commits into
reactjs:mainfrom
EricCote:fixLinkDocs
Dec 16, 2024
Merged

Fix stylesheet precedence example#7235
gaearon merged 2 commits into
reactjs:mainfrom
EricCote:fixLinkDocs

Conversation

@EricCote

Copy link
Copy Markdown
Contributor

The precedence prop documentation for <style> and <link> was fixed last May in this pull request:
#6908
(Precedence is now using any name, instead of "high", "medium" or "low")

But the code example accompanying the <link> documentation was not fixed.

There was even a comment in the doc about the fact that precedence didn't seem to work. -- It did work, it's just that it's more of a grouping behavior than an ordering behavior. So you need at least 3 <link> to visibly show that grouping behavior.

@vercel

vercelBot commented Oct 20, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
19-react-dev✅ Ready (Inspect)Visit Preview💬 Add feedbackDec 16, 2024 3:58pm
react-dev✅ Ready (Inspect)Visit PreviewDec 16, 2024 3:58pm

@github-actions

github-actionsBot commented Oct 20, 2024

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! 🙌

@gaearongaearon left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

omg thank you, this was so confusing

Sign up for freeto 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.

3 participants

@EricCote@gaearon@facebook-github-bot