-
Notifications
You must be signed in to change notification settings - Fork 109
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
Add support for ELK layout engine #458
Comments
@ahus1 I think this is feature request for you. |
I had a look during the holidays, and this is what I found:
In the meantime:
As a follow-up, please create an issue in the https://github.com/asciidoctor/asciidoctor-intellij-plugin. Still I don't think I'll include it until the service loader mechanism is working also in custom class loaders. It might be possible to call the registration manually, still I'd like to avoid that. Let's see what the ELK project decides. |
@ahus1 not sure what I can do about the license situation. If I understand it correctly, we would need to choose between either the LGPL version that contains Ditaa support or the EPL version which contains ELK support. |
For better layout of PlantUML diagrams, I mostly use ELK (Eclipse Layout Kernel). As ELK is not part of the official PlantUML distribution, using asciidoctor-diagram cannot render such diagrams. This issue can easily circumvented when e.g. using the asciidocotr Maven plugin by simply adding the following ELK libraries as a dependency to the plugin.
However, when e.g. using the https://github.com/asciidoctor/asciidoctor-intellij-plugin this way of providing additional libraries is not possible. Following the ELK discussions and timeline around PlantUML, I don't see that it will be integrated soon into PlantUML itself.
Therefore, it would be great if asciidoctor-diagram could bring the dependencies to ELK itself and thus support a third layouting option besides GraphViz and Smetana for many tools built on top of asciidoctor-diagram.
The text was updated successfully, but these errors were encountered: