Skip to content

apps/ui: Add widget toolbar overlay and note delete action#3409

Merged
youknowriad merged 2 commits into
trunkfrom
codex/implement-widget-toolbar
May 9, 2026
Merged

apps/ui: Add widget toolbar overlay and note delete action#3409
youknowriad merged 2 commits into
trunkfrom
codex/implement-widget-toolbar

Conversation

@youknowriad
Copy link
Copy Markdown
Contributor

Summary

  • Added a shared desk widget toolbar overlay rendered in tldraw’s InFrontOfTheCanvas slot.
  • Added a widget-level toolbar contract so individual widgets can supply their own controls.
  • Wired the sticky note widget to expose its tone picker through the new toolbar surface.
  • Added a shared trailing separator and remove button so the current widget can be deleted from the toolbar.

Testing

  • npx eslint --fix on the modified TypeScript files.
  • npm run typecheck.
  • npm test -- apps/ui/src/ui-desks/widgets/toolbar-selection.test.ts apps/ui/src/ui-desks/user-desk/tldraw-adapter.test.ts.

@wpmobilebot
Copy link
Copy Markdown
Collaborator

wpmobilebot commented May 9, 2026

📊 Performance Test Results

Comparing 042daa8 vs trunk

app-size

Metric trunk 042daa8 Diff Change
App Size (Mac) 1408.80 MB 1408.80 MB +0.00 MB ⚪ 0.0%

site-editor

Metric trunk 042daa8 Diff Change
load 1464 ms 1489 ms +25 ms ⚪ 0.0%

site-startup

Metric trunk 042daa8 Diff Change
siteCreation 8100 ms 8100 ms 0 ms ⚪ 0.0%
siteStartup 4921 ms 4936 ms +15 ms ⚪ 0.0%

Results are median values from multiple test runs.

Legend: 🟢 Improvement (faster) | 🔴 Regression (slower) | ⚪ No change (<50ms diff)

@youknowriad youknowriad changed the title Add widget toolbar overlay and note delete action apps/ui: Add widget toolbar overlay and note delete action May 9, 2026
@youknowriad youknowriad force-pushed the codex/implement-widget-toolbar branch from 004cb1c to efb4b06 Compare May 9, 2026 13:09
@youknowriad youknowriad merged commit fe5145f into trunk May 9, 2026
8 of 10 checks passed
@youknowriad youknowriad deleted the codex/implement-widget-toolbar branch May 9, 2026 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants