Skip to content

replacing <img> with <Image/>#8439

Closed
Riya61018 wants to merge 1 commit into
reactjs:mainfrom
Riya61018:new-branch
Closed

replacing <img> with <Image/>#8439
Riya61018 wants to merge 1 commit into
reactjs:mainfrom
Riya61018:new-branch

Conversation

@Riya61018
Copy link
Copy Markdown

fixes #4382

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

@MaxwellCohen
Copy link
Copy Markdown

I have some concerns about this PR. The component is Next.js (and some other meta frameworks) specific component not part of React, so React documentation should not be updated to tell using the . There is a need to update the code of react.dev to leverage component where it is not leveraged, so that this repo can be a good demo of best practices around Next and React.

@Riya61018
Copy link
Copy Markdown
Author

Should I close this pr?

@MaxwellCohen
Copy link
Copy Markdown

I would, this PR does not fix the issue with the mdx component

@Riya61018 Riya61018 closed this May 17, 2026
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.

Add next/image component as MDX component and convert img tags to Image component

2 participants