Skip to content

Improve shared diagram rendering and honor authored styles - #1154

Merged
raphael merged 1 commit into
mainfrom
diagram-visuals
Nov 10, 2026
Merged

raphael merged 1 commit into
mainfrom
diagram-visuals

Conversation

@raphael

@raphael raphael commented Nov 10, 2026

Copy link
Copy Markdown
Member

Model now draws the same flat, readable diagram in the editor, static page and exported SVG. Previously, duplicate renderer styles added shadows and mixed fonts, while standalone exports relied on fonts installed by the reader. Some existing author choices also disappeared: metadata/description visibility and icons were ignored, deployment-boundary defaults overrode explicit colors, and relationship labels dropped technology and ignored their configured wrapping width.

The shared renderer now uses one style definition, thinner borders and clearer defaults. It honors those existing style fields while preserving authored shapes, colors, dimensions and the existing layout rules. Relationship labels use the author's width and include technology when supplied. SVGs carry the same local Inter font faces used during measurement, together with their license, so they display consistently without a network connection. This adds roughly 136 KB to each SVG.

A browser regression serves a typed design through the real editor, exports it, and opens the file in a fresh browser. It checks actual icon decoding, standalone fonts and license, an explicitly sized Cylinder, hidden metadata/description, custom deployment-boundary colors, and 120px relationship wrapping. Existing text-fit, collision, component-boundary and manual-edit checks also pass. The basic and all-shapes examples were inspected before and after. Saved-coordinate and incremental-layout behavior are unchanged.

@raphael
raphael merged commit 28dd627 into main Nov 10, 2026
3 checks passed
@raphael
raphael deleted the diagram-visuals branch November 10, 2026 07:32
@raphael raphael mentioned this pull request Nov 10, 2026
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