Support callbacks in React

[jprochazk]

• Part of Bring React package to feature parity with JS package #10495

pixi run js-build-all
pixi run yarn --cwd rerun_js workspaces run pack

Produces rerun_js/web-viewer/web-viewer.tar.gz and rerun_js/web-viewer-react/web-viewer-react.tar.gz

These can be installed into e.g. https://github.com/rerun-io/web-viewer-react-example by running:

npm i # base dependencies
npm i ../rerun/rerun_js/web-viewer/web-viewer.tar.gz
npm i ../rerun/rerun_js/web-viewer-react/web-viewer-react.tar.gz

Afterwards run the example as normal. You can find the new callback props available on the WebViewer component, e.g. onPlay, onTimeUpdate, etc.

Implementation details

I felt bad about adding another layer of stuff that anyone working on events has to deal with, so this requires no effort whatsoever to maintain.

The callback prop types are generated automatically from @rerun-io/web-viewer event types. When a callback prop is registered, the prop name is transformed to the event name (remove on prefix, snake_case it), which is then directly registered on the underlying WebViewer instance. This way we don't need to maintain a list of events on the React side, it stays up to date automatically as new events are added in the non-React package.

[github-actions]

Web viewer built successfully. If applicable, you should also test it:

• I have tested the web viewer

Result	Commit	Link	Manifest
✅	e4cde28	https://rerun.io/viewer/pr/10518	+nightly +main

^{Note: This comment is updated whenever you push a commit.}

[jprochazk]

I thought there was some kind of performance regression in the React package, but I probably just built it in debug mode 😬. Can't repro anymore, so this is ready

[Wumpf]

this prop name to event name transformation could be a bit brittle though, no? Sounds like we should leave a note to keep those names carefully in sync
Otherwise really nice to not have those duplicated once more!

[jprochazk]

this prop name to event name transformation could be a bit brittle though, no?

Yes, should add a note about making sure the event names are snake_cased versions of the same events on the Rust side