Skip to content

create-root#50

Merged
moshfiqrony merged 6 commits into
reactjs:mainfrom
nafistiham:client-api-create-root
Jun 8, 2023
Merged

create-root#50
moshfiqrony merged 6 commits into
reactjs:mainfrom
nafistiham:client-api-create-root

Conversation

@nafistiham

Copy link
Copy Markdown
Collaborator

No description provided.

@github-actions

github-actions Bot commented Jun 1, 2023

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

@nafistiham nafistiham marked this pull request as ready for review June 8, 2023 09:59

@moshfiqrony moshfiqrony 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.

The translation is legit. I had some confusion so asked about it.

* The first time you call `root.render`, React will clear all the existing HTML content inside the React root before rendering the React component into it.
* আপনি যখন প্রথম বারের মত `root.render` কল করবেন, React এর রুটের ভিতরে কম্পোনেন্ট রেন্ডার করবার আগে থাকা সব HTML কনটেন্ট মুছে ফেলবে।

* If your root's DOM node contains HTML generated by React on the server or during the build, use [`hydrateRoot()`](/reference/react-dom/client/hydrateRoot) instead, which attaches the event handlers to the existing HTML.

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.

Bhai ei line ta ki intentionally translate korenni?

Comment thread src/content/reference/react-dom/client/createRoot.md Outdated
Comment thread src/content/reference/react-dom/client/createRoot.md Outdated

@moshfiqrony moshfiqrony 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.

LGTM

@moshfiqrony moshfiqrony added this pull request to the merge queue Jun 8, 2023
Merged via the queue into reactjs:main with commit dedf127 Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants