Skip to content

Remove unused mermaid flavoured flow code#63

Merged
Matthbo merged 5 commits intomasterfrom
issue/62-remove-html-to-svg-conversion-code
Mar 10, 2026
Merged

Remove unused mermaid flavoured flow code#63
Matthbo merged 5 commits intomasterfrom
issue/62-remove-html-to-svg-conversion-code

Conversation

@Matthbo
Copy link
Copy Markdown
Member

@Matthbo Matthbo commented Mar 2, 2026

No description provided.

@Matthbo Matthbo linked an issue Mar 2, 2026 that may be closed by this pull request
@Matthbo Matthbo self-assigned this Mar 2, 2026
@Matthbo Matthbo requested a review from mhdirkse March 2, 2026 16:02
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Notice how the whitespace on newlines have changed, this is to be more consistent with lines that don't start with whitespaces anyway.
Sadly it doesn't look nice in code but this wont matter much when its used in the browser.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed filter that wont be necessary with the new flow code

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wasn't sure if this is the right thing to do, saving the true text content won't be useful for generating the svg so I fixed it this way.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Uses the flow code generated SVG text elements and adds the positioning attributes instead of generating new elements

Comment thread projects/frank-config-layout/src/lib/model/text.ts Outdated
Comment thread projects/frank-config-layout/src/lib/graphics/svg-generator.ts
Comment thread projects/frank-config-layout/src/lib/mermaid2svg.spec.ts Outdated
@Matthbo Matthbo requested a review from mhdirkse March 4, 2026 09:01
@Matthbo Matthbo merged commit 2821b85 into master Mar 10, 2026
1 check passed
@Matthbo Matthbo deleted the issue/62-remove-html-to-svg-conversion-code branch March 10, 2026 09:09
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.

Remove html to svg conversion code

2 participants