feat: Change return type of render to ReactNode

[yroc92]

For compatibiility with React 18, we need the return type of render to be ReactNode instead of ReactElement.

Closes #74

[bvaughn]

Shouldn't this be the opposite? Or is it only Function components that should now be ReactElement instead of ReactNode? I haven't been keeping up with that.

[yroc92]

@bvaughn This might answer your question: https://stackoverflow.com/questions/54544483/should-a-components-render-method-have-return-type-react-reactnode-or-jsx-eleme

[bvaughn]

Hmm. It seems that for function components the change required is exactly the opposite of this (see bvaughn/react-resizable-panels#256)

I guess the Stack Overflow you linked to mentions this:

TS render types are currently limited to:
Class component: ReactNode (wider than permitted by React)
Function component: JSX.Element | null (more restrictive than React)

Looking at the TypeScript types in DefinitelyTyped, it does look like

render(): ReactNode

Kind of a pain in the ass 😄

[bvaughn]

I guess this change makes sense 🤷🏼 I dunno

[bvaughn]

Change published in react-virtualized-auto-sizer@1.0.21

---

❤️ → ☕ givebrian.coffee

[yroc92]

Kind of a pain in the ass 😄

I agree 100%. Thank you!