You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, the DAG output implementation for HTML is stand-alone, and needs to be maintained separately. The Mermaid DAG has been improved quite a bit recently (#4070). Rather than trying to maintain the two codebases separately, we could just reuse the Mermaid output for the HTML file.
Basically, if HTML is requested - create the Mermaid DAG output file and embed it within a static HTML template. The template should include the relevant Mermaid javascript dependencies and script blocks, as described in the Mermaid docs.
This should give us nicer functionality and less maintenance burden.
Note that the HTML DAG config scopes should probably still stay in place, but would then just forward on to the Mermaid DAG generation code.
With the HTML renderer I think we could go much farther than simply wrapping around the mermaid diagram. You can attach JS event handlers to nodes in a mermaid diagram, so we could make it interactive, i.e. click to expand/collapse sub-workflows. Also add hyperlinks for operators to the Nextflow docs and processes to their source code
Currently, the DAG output implementation for HTML is stand-alone, and needs to be maintained separately. The Mermaid DAG has been improved quite a bit recently (#4070). Rather than trying to maintain the two codebases separately, we could just reuse the Mermaid output for the HTML file.
Basically, if HTML is requested - create the Mermaid DAG output file and embed it within a static HTML template. The template should include the relevant Mermaid javascript dependencies and script blocks, as described in the Mermaid docs.
This should give us nicer functionality and less maintenance burden.
Note that the HTML DAG config scopes should probably still stay in place, but would then just forward on to the Mermaid DAG generation code.
👉🏻 Related to #4315
The text was updated successfully, but these errors were encountered: