Skip to content

[5/6] Add a dedicated asset generation toolbar - #161

Open
dazzatronus wants to merge 1 commit into
derk/sdk-generation-pending-statefrom
derk/sdk-generation-toolbar
Open

[5/6] Add a dedicated asset generation toolbar#161
dazzatronus wants to merge 1 commit into
derk/sdk-generation-pending-statefrom
derk/sdk-generation-toolbar

Conversation

@dazzatronus

@dazzatronus dazzatronus commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

Prompt generation needs a dedicated editing surface while asset properties and clip timing remain reachable.

A third toolbar mode contains the inline prompt, Generate/Retry state and error feedback. The controller routes mode changes and keeps generation markers synchronised across every mounted toolbar.

This is 699 lines because the toolbar DOM and state synchronisation, controller routing, shared mode CSS and interaction tests share one mount/dispose contract. The decisions sit in generate-toolbar.ts and ui-controller.ts; mirrored toolbar markup, CSS and tests are integration churn. Splitting at those boundaries leaves either an unreachable pane or listeners without an owner.

Verify: npm run verify:ci

@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from fdbe0b6 to 95e2932 Compare August 25, 2026 00:31
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 95e2932 to c186d6c Compare August 25, 2026 01:28
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch 2 times, most recently from 32bf05e to 7e99d7e Compare August 25, 2026 04:41
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 7e99d7e to 315f2ed Compare August 25, 2026 06:02
@dazzatronus dazzatronus changed the title [4/6] Add a dedicated asset generation toolbar [5/6] Add a dedicated asset generation toolbar Aug 25, 2026
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 315f2ed to 1ae972d Compare August 25, 2026 06:18
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 1ae972d to 2825b7b Compare August 25, 2026 06:31
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 2825b7b to ba79fc8 Compare August 25, 2026 06:58
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from ba79fc8 to 63996a7 Compare August 25, 2026 07:18
@dazzatronus
dazzatronus force-pushed the derk/sdk-generation-toolbar branch from 63996a7 to e5a2134 Compare August 25, 2026 07:32
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.

1 participant