-
Notifications
You must be signed in to change notification settings - Fork 123
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
Implement Disqus plugin #1449
Implement Disqus plugin #1449
Conversation
@ang-zeyu Hi Zeyu, this PR is ready for your review :-) if everything looks fine, I will proceed to update the test. |
@ang-zeyu The PR is ready for your review :-). I've made the changes you suggested. I've also ensured that the redirecting of the path are handled accordingly. Although i'm not sure if there might be any edge cases that I missed out (hopefully not). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple of nits and one issue / clarification:
|
||
<box type="tip"> | ||
|
||
It may be a good idea to create a [layout]({{ baseUrl }}/userGuide/tweakingThePageStructure.html#layouts) with `<disqus/>` in it, so that you won't have to repeat it on every page. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
{{ baseUrl }}/userGuide/tweakingThePageStructure.html#layouts
I specified it as .html
currently as our .md to .html conversion currently can't handle links with #.
I will make a PR to update the implementation and change this to .md
afterwards :-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lgtm!
What is the purpose of this pull request?
This PR resolves #481.
Overview of changes:
Setup Disqus plugin with lazy loading using IntersectionObserver API.
Anything you'd like to highlight / discuss:
Testing instructions:
Add
<disqus/>
to any page andmarkbind serve
.You should be able to see the disqus comments section (it should load only when you scroll near it).
Proposed commit message: (wrap lines at 72 characters)
Implement Disqus plugin.
Users may want to allow readers to comment on content in the website.
This can be an important feature for educational websites as it enables
better student interaction.
Let's integrate Disqus into MarkBind to enable this commenting feature.
Checklist: ☑️