adebb680e5
When we went from overrides-based to component based UI customisation APIs, we didn't do the toolbar because it had some significant extra complexity around overflowing the contents of the menu into the dropdown. This is really hard to do at render-time with react - you can't introspect what a component will return to move some of it into an overflow. Instead, this diff runs that logic in a `useLayoutEffect` - we render all the items into both the main toolbar and the overflow menu, then in the effect (or if the rendered components change) we use CSS to remove the items we don't need, check which was last active, etc. Originally, I wasn't really into this approach - but i've actually found it to work super well and be very reliable. ### Change Type - [x] `major` — Breaking change - [ ] `dependencies` — Changes to package dependencies[^1] - [ ] `documentation` — Changes to the documentation only[^2] - [ ] `tests` — Changes to any test code only[^2] - [ ] `internal` — Any other changes that don't affect the published package[^2] - [ ] I don't know [^1]: publishes a `patch` release, for devDependencies use `internal` [^2]: will not publish a new version ### Test Plan 1. Test the toolbar at many different sizes with many different 'active tools' --------- Co-authored-by: Steve Ruiz <steveruizok@gmail.com> |
||
---|---|---|
.. | ||
export-snapshots.spec.ts-snapshots | ||
fixtures | ||
test-canvas-events.spec.ts-snapshots | ||
export-snapshots.spec.ts | ||
test-actions-menu.spec.ts | ||
test-camera.spec.ts | ||
test-canvas-events.spec.ts | ||
test-clipboard.spec.ts | ||
test-context-menu.spec.ts | ||
test-focus.spec.ts | ||
test-help-menu.spec.ts | ||
test-kbds.spec.ts | ||
test-main-menu.spec.ts | ||
test-navigation-panel.spec.ts | ||
test-page-menu.spec.ts | ||
test-routes.spec.ts | ||
test-shapes.spec.ts | ||
test-smoke.spec.ts | ||
test-style-panel.spec.ts | ||
test-text.spec.ts | ||
test-toolbar.spec.ts |