Skip to content

fix(diagrams): enable sequence annotations#6

Merged
auspham merged 1 commit into
mainfrom
austinpham/fix-sequence-diagram-annotation
Jul 15, 2026
Merged

fix(diagrams): enable sequence annotations#6
auspham merged 1 commit into
mainfrom
austinpham/fix-sequence-diagram-annotation

Conversation

@auspham

@auspham auspham commented Jul 15, 2026

Copy link
Copy Markdown
Owner

Summary

  • Recognize Mermaid sequence participants, messages, arrows, and notes as annotation targets while preserving drag-to-pan.
  • Assign stable target keys so diagram comment anchors survive Mermaid re-renders and page reloads.
  • Add a browser regression that sends a sequence comment and verifies its anchor re-resolves after reload.

Testing

  • PATH=/home/austin/.nvm/versions/node/v22.13.0/bin:$PATH npm test --silent

Copilot-Session: d2d2e8e1-3359-4935-a94e-0a1906d97b56
Signed-off-by: Austin (Ngoc Thang) Pham <austinpham@microsoft.com>
@auspham auspham merged commit 2fb3b7d into main Jul 15, 2026
4 checks passed
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