feat(drawnix): support nested summary rendering #4

Merged

jesse merged 6 commits from feat/drawnix-mind-summarize-topics into master 2026-05-04 21:22:17 +02:00

Conversation 0 Commits 6 Files changed 24 +3024 -71

jesse commented 2026-05-04 16:45:23 +02:00

Owner

Copy link

Summary

• Keeps the Summary action visible with clearer eligibility behavior and allows nested/overlapping summary creation when selecting non-abstract sibling topics.
• Styles summary nodes from their source topic and refines the curly-brace rendering geometry.
• Adds footprint-based nested-summary layout so outer summaries clear inner summary braces and nodes, with docs and issue/session notes.

Tests

• Not run during PR creation.

## Summary - Keeps the Summary action visible with clearer eligibility behavior and allows nested/overlapping summary creation when selecting non-abstract sibling topics. - Styles summary nodes from their source topic and refines the curly-brace rendering geometry. - Adds footprint-based nested-summary layout so outer summaries clear inner summary braces and nodes, with docs and issue/session notes. ## Tests - Not run during PR creation.

jesse added 5 commits 2026-05-04 16:45:23 +02:00

feat(drawnix): make summary button always visible with per-reason tooltips 5917427682

feat(drawnix): style summary node to match source siblings ... 4025e36736

After insertAbstract creates an abstract node with the library's outlined
default style, snapshot the tree, diff for the new abstract, and patch its
fill/strokeColor from the first selected sibling so the summary visually
integrates with the surrounding mind theme.

The patch is best-effort: only explicit (non-default) values are propagated,
the inserted node is located via a tree-wide diff to avoid relying on parent
weakmaps, and getStrokeColorByElement failures are swallowed. The style
patch produces a separate undo step from the insertion itself; grouping
would require batching primitives @plait/core does not currently expose.

Coverage: summary-style.ts 100%, mind-actions.tsx 90.9%.

fix(drawnix): summary node fill falls back to theme-resolved value ... 64af701488

Regular mind nodes don't carry an explicit fill — they inherit it from
the active theme. The previous patch only copied explicit fields, so
the inserted abstract kept the library's outlined default. Resolve
fill via getFillByElement, mirroring the existing strokeColor path.

fix(drawnix): refine summary brace visual 80c217d9a6

feat(drawnix): support nested summary rendering aaf532c62f

jesse added 1 commit 2026-05-04 17:38:40 +02:00

fix(drawnix): satisfy summary style lint 08adf9250b

jesse merged commit 02dc32efd0 into master 2026-05-04 21:22:17 +02:00

jesse referenced this pull request from a commit 2026-05-04 21:22:17 +02:00

Merge pull request 'feat(drawnix): support nested summary rendering' (#4) from feat/drawnix-mind-summarize-topics into master

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

jesse/mindnest!4

No description provided.