-
-
Notifications
You must be signed in to change notification settings - Fork 32
Conversation
release: 0.1.2
…yasin docs: add aiyasin as a contributor
fix(gist): fixed the lost of indentation
…latanado docs: add aplatanado as a contributor for code
…w717 docs: add ktw717 as a contributor for code, doc
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.
Hi @OnlyFor.
Thank you very much for your contributions. I left some comments on this PR.
Also, other things we will need to update:
- Update the
README.md
file and add the required information in the features of these new comment systems. - Update the
Theme Documentation basics
article, on the configuration section, about these newly added comment systems.
exampleSite/config.toml
Outdated
[params.page.comment.waline] | ||
enable = false | ||
#js = "https://cdn.jsdelivr.net/npm/@waline/client@latest" | ||
js = "https://cdn.jsdelivr.net/npm/@waline/client/dist/Waline.min.js" |
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.
Can you move please this parameter in the assets/data/cdn/jsdelivr.yml
file?
We have a single place for the CDN files.
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.
Done.
exampleSite/config.toml
Outdated
dark = 'auto' | ||
[params.page.comment.twikoo] | ||
enable = false | ||
js = "https://cdn.jsdelivr.net/npm/[email protected]/dist/twikoo.all.min.js" |
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.
I think the properties should be indented at the same level, they're all referenced to the params.page.comment.twikoo
group.
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.
Done.
layouts/partials/comment.html
Outdated
{{- $twikoo := $comment.twikoo | default dict -}} | ||
{{- if $twikoo.enable -}} | ||
<div id="twikoo"></div> | ||
<script src="{{ $twikoo.js }}"></script> |
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.
Can you use something like this?
CodeIT/layouts/partials/comment.html
Lines 19 to 31 in a74cdcf
{{- /* Gitalk Comment System */ -}} | |
{{- $gitalk := $comment.gitalk | default dict -}} | |
{{- if $gitalk.enable -}} | |
<div id="gitalk" class="comment"></div> | |
{{- $source := $cdn.gitalkCSS | default "lib/gitalk/gitalk.css" -}} | |
{{- dict "Source" $source "Minify" true "Fingerprint" $fingerprint | dict "Scratch" .Scratch "Data" | partial "scratch/style.html" -}} | |
{{- $source := $cdn.gitalkJS | default "lib/gitalk/gitalk.min.js" -}} | |
{{- dict "Source" $source "Fingerprint" $fingerprint | dict "Scratch" .Scratch "Data" | partial "scratch/script.html" -}} | |
{{- $commentConfig = dict "id" .Date "title" .Title "clientID" $gitalk.clientId "clientSecret" $gitalk.clientSecret "repo" $gitalk.repo "owner" $gitalk.owner "admin" (slice $gitalk.owner) | dict "gitalk" | merge $commentConfig -}} | |
<noscript> | |
Please enable JavaScript to view the comments powered by <a href="https://github.com/gitalk/gitalk"></a>Gitalk</a>. | |
</noscript> | |
{{- end -}} |
We have the script partial page for this, and we can use also the file from the jsdelivr.yml
file. Also, as a default value, it would be nice to have a local copy of the twikoo/waline scripts files. They are stored in this location: assets/lib
.
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.
I tried...but I only learned javascript for a few weeks...I hope you can help adjust if necessary
layouts/posts/single.html
Outdated
<span id="{{ .RelPermalink }}" class="leancloud_visitors" data-flag-title="{{ .Title }}"> | ||
<i class="far fa-eye fa-fw"></i> <span class=leancloud-visitors-count></span> {{ T "views" }} | ||
</span> | ||
<a href="#comments" id="post-meta-vcount" title="{{ T `viewComments` }}"> |
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.
The indentation should be the same as is on the 65th line.
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.
May be done ?
@@ -24,3 +24,5 @@ [email protected] https://github.com/metowolf/MetingJS | |||
[email protected] https://github.com/gitalk/gitalk | |||
[email protected] https://valine.js.org/ | |||
[email protected] https://github.com/osano/cookieconsent | |||
[email protected] https://github.com/imaegoo/twikoo |
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.
@OnlyFor did you use some packaging tool or did you manually update this file? I'm asking this because I don't know how this file was generated ))
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.
I……manually update this file……
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.
No worries. I just asked. I thought maybe there is some package manager tool.
@@ -563,6 +563,17 @@ Please open the code block below to view the complete sample configuration :(far | |||
label = "" | |||
lightTheme = "github-light" | |||
darkTheme = "github-dark" | |||
# {{< link "https://waline.js.org/en/" "Waline" >}} comment config |
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.
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.
Done.
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.
Good job. Thank you !!!
@@ -24,3 +24,5 @@ [email protected] https://github.com/metowolf/MetingJS | |||
[email protected] https://github.com/gitalk/gitalk | |||
[email protected] https://valine.js.org/ | |||
[email protected] https://github.com/osano/cookieconsent | |||
[email protected] https://github.com/imaegoo/twikoo |
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.
No worries. I just asked. I thought maybe there is some package manager tool.
@all-contributors please add @OnlyFor for code, docs |
I've put up a pull request to add @OnlyFor! 🎉 |
No description provided.