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

👽 Update plugin to be compatible with mermaid V10 #30

Merged
merged 1 commit into from
Mar 9, 2023
Merged

👽 Update plugin to be compatible with mermaid V10 #30

merged 1 commit into from
Mar 9, 2023

Conversation

corentinleberre
Copy link
Contributor

@corentinleberre corentinleberre commented Mar 7, 2023

Hi,

I updated the plugin due to this issue.
Now it support mermaid V10.

@corentinleberre corentinleberre changed the title 👽 Update plugin to be compatible with mermaid V10 WIP: 👽 Update plugin to be compatible with mermaid V10 Mar 7, 2023
@corentinleberre corentinleberre changed the title WIP: 👽 Update plugin to be compatible with mermaid V10 👽 Update plugin to be compatible with mermaid V10 Mar 7, 2023
@corentinleberre corentinleberre marked this pull request as ready for review March 7, 2023 21:54
src/plugin.js Outdated
Comment on lines 21 to 23
hook.doneEach(() => {
mermaid.run(mermaidConf);
});

Choose a reason for hiding this comment

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

// Invoked one time after rendering the initial page
    hook.ready(function () {
      // ...
    });

This should be enough to render all diagrams at once.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You're right! I modified my commit.

@Leward
Copy link
Owner

Leward commented Mar 9, 2023

Thank you for the MR. I will merge and tag a new version.

@Leward Leward merged commit f59b4f2 into Leward:master Mar 9, 2023
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.

3 participants