Sanjaiyan Parthipan

Hi, I thought we can incrementally adopt to **React 18** which give easy migration process too 😊 ```jsx if (React.version.includes("18.")) { if (process.env.NODE_ENV === "production") { const { hydrateRoot }...

> @sanjaiyan-dev That falls under the realm of > > > Problem: hacky code, slightly increases bundle size, and harder to maintain in the long run (refactoring hazards) > >...

> > Hi, I thought to use new [`useInsertionEffect`](https://reactjs.org/docs/hooks-reference.html#useinsertioneffect) for `` component hence it used to toggle CSS efficiently where previously we used `useLayoutEffect`. > > ```js > > //...

> Closing [as noted above](https://github.com/vercel/next.js/pull/39356#discussion_r944158937) this change doesn't seem necessary. Thanks for investigating though. 👍🙌

> > Using component from next/script for inline colour mode script. > > Can you elaborate what behavior you're seeing without this change, what behavior the change results in, and...

> > Sorry if I made any mistakes raised_handsblush > > I didn't mean to imply that. I was wondering what this change is doing in the first place or...

> > Sorry if I made any mistakes raised_handsblush > > I didn't mean to imply that. I was wondering what this change is doing in the first place or...

@gaearon Sorry sir, actually I thought dynamically loading the component should improve load speed hence that particular component is conditionally loaded (according to screen size) I am sorry if I...

> how did you verify that something's actually improved? can you share your methodology? @gaearon , I thought loading of `` component is conditional according to screen size so by...

Hope I ran the `next:lint` correctly [as requested](https://github.com/reactjs/reactjs.org/pull/4805#issuecomment-1179294382) :) (@harish-sethuraman )