Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plantuml svg loader #800

Merged
merged 9 commits into from Jan 24, 2022
Merged

Plantuml svg loader #800

merged 9 commits into from Jan 24, 2022

Conversation

erwin-faceit
Copy link
Contributor

This PR gives the possibility to use an SVG loader instead of an using an img tag when using PlantUML. The reason for this is that the image tag does not support links or other responsive actions in the generated UML diagram.

By default, nothing changes. If you want this PR to be enabled, you'll need to change the configuration like this

[params.plantuml]
enable = true
svg = true

Setting the params.plantuml.svg property to true will add an additional JavaScript file (svg-loader.min.js), and will write svg tags instead of img tags.

@LisaFC
Copy link
Collaborator

LisaFC commented Jan 20, 2022

Ooh, this looks very helpful. Do you want to also add some instructions to https://www.docsy.dev/docs/adding-content/diagrams-and-formulae/#uml-diagrams-with-plantuml so users will know how to enable this feature?

@erwin-faceit
Copy link
Contributor Author

@LisaFC: A small line perhaps, but is this enough, or would you prefer a more extensive writeup about this?

@LisaFC
Copy link
Collaborator

LisaFC commented Jan 24, 2022

No, that's perfect! I'm sure someone will let us know if they want more information.

@LisaFC LisaFC merged commit 3e2cfeb into google:master Jan 24, 2022
@deining
Copy link
Collaborator

deining commented Feb 1, 2022

I just realized that this PR sets submodule bootstrap back to v4.6.0, while we were at v4.6.1 already. Is/was this intended?

@erwin-faceit
Copy link
Contributor Author

erwin-faceit commented Feb 1, 2022

No, this was not intended! Did I merge the main branch incorrectly? I don't see anything related to bootstrap in the changed files?

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.

None yet

4 participants