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

fix(comment/waline): cannot read property ‘insertBefore’ of null #1103

Merged
merged 2 commits into from
Dec 22, 2024

Conversation

fsj2009yx
Copy link
Contributor

When I use the waline compoment to build comment section for my blog,the comment data couldn't pull and raise Error "Cannot read property ‘insertBefore’ of null".then I get the answer from #1044 and guess that it's due to the Katex renders before the waline compoments.
Add "waline-containertoignoredClasses` could work.But changing the element to render KateX may be used for most of situations,that allows each comment system to use their own Latex renderer.

Hope this helps you!

@fsj2009yx
Copy link
Contributor Author

In this way KateX works still fine in comment section,At least it did in waline.
91fc0606f29110ad6149fda7ccf0b351

Copy link
Owner

@CaiJimmy CaiJimmy left a comment

Choose a reason for hiding this comment

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

Thanks!

@CaiJimmy CaiJimmy changed the title Fix :the problem that comment data couldn't be pulled successfully fix(comment/waline): cannot read property ‘insertBefore’ of null Dec 22, 2024
@CaiJimmy CaiJimmy merged commit 0991f61 into CaiJimmy:master Dec 22, 2024
5 checks passed
@CaiJimmy CaiJimmy linked an issue Dec 22, 2024 that may be closed by this pull request
Kouni pushed a commit to Kouni/hugo-theme-stack that referenced this pull request Feb 8, 2025
…Jimmy#1103)

* fix: resolve issue with comments not displaying due to KaTeX and Waline render order

* Fix the problem that comment data couldn't be pulled successfully and raise Error "Cannot read property ‘insertBefore’ of null""
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.

关于最新版与Waline评论冲突问题
2 participants