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

Markdown: Workaround for double wrap hook #2719

Merged

Conversation

RunDevelopment
Copy link
Member

This is a workaround for #2716.

The problem in #2716 is that Markdown's wrap hook function might be added multiple times because the language might be reloaded. I did not account for that when writing the function, so this is a fix for that. However, it does not solve the underlying issue.

This workaround should be kept even after #2716 was fixed because other plugins might modify env.content before Markdown and add custom HTML.

@github-actions
Copy link

JS File Size Changes (gzipped)

A total of 1 files have changed, with a combined diff of +7 B (+0.4%).

file master pull size diff % diff
components/prism-markdown.min.js 1.76 KB 1.76 KB +7 B +0.4%

Generated by 🚫 dangerJS against c85b411

@RunDevelopment RunDevelopment merged commit 2b355c9 into PrismJS:master Jan 29, 2021
@RunDevelopment RunDevelopment deleted the markdown-double-wrap-workaround branch January 29, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant