fix: declare children prop explicitly for class components

[adidahiya]

Fixes #5250

Follow-up to #5258

Changes proposed in this pull request:

Audit all remaining usage of props.children. Where components do use children, we now declare that prop type explicitly for compatibility with @types/react@18.

[blueprint-bot]

fix: declare children prop explicitly for class components

Previews: documentation | landing | table | demo

[adidahiya]

Suggested change

/** Drawer contents. */

[adidahiya]

this should probably go into IPopoverSharedProps

[blueprint-bot]

address self-review

Previews: documentation | landing | table | demo

[blueprint-bot]

fix test type errors

Previews: documentation | landing | table | demo

[andreash]

I am using @types/react 18.0.21 and @blueprintjs/table 4.7.4. I am still getting a type-script error in vscode for Column components:

'Column' cannot be used as a JSX component.
  Its instance type 'Column' is not a valid JSX element.
    The types returned by 'render()' are incompatible between these types.
      Type 'ReactNode' is not assignable to type 'false | Element'.
        Type 'string' is not assignable to type 'false | Element'.ts(2786)

Any idea?

Code example:

ReactDOM.render(
    <HotkeysProvider>
        <Table2 numRows={5}>
            <Column />
            <Column />
            <Column />
        </Table2>
    </HotkeysProvider>,
    document.querySelector("#app"),
);