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

Site-wide disqus comments not activated via _config.yml? #94

Closed
brainstorm opened this issue Sep 26, 2014 · 3 comments
Closed

Site-wide disqus comments not activated via _config.yml? #94

brainstorm opened this issue Sep 26, 2014 · 3 comments

Comments

@brainstorm
Copy link
Contributor

Hi Michael,

Terrific job with So-simple-theme, I love it 👍

I've recently migrated my blog from WP to Jekyll and your theme, but apparently I cannot enable comments globally in _config.yml, instead I have to say comments: true on every blog post:

https://github.com/brainstorm/brainstorm.github.io/commit/3a8e9e4999f

Any ideas why?

@mmistakes
Copy link
Owner

I went with a more granular approach.

You can enable it in a few ways. Edit the post.html _layout and scripts.html _includes and remove the conditional if statement around the Disqus div and script. That will effectively turn on Disqus for every page.

Or you can modify the conditionals to now show Disqus when you add something like comments: false in a post's Front Matter YAML if there are a couple of pages you don't want comments on.

@brainstorm
Copy link
Contributor Author

Thanks @mmistakes, I went for this option:

https://github.com/brainstorm/brainstorm.github.io/commit/cc10b7f7b3385

I think it's the one what gives most flexibility (and granularity). Would you like me to prepare a pullrequest? Are the changes acceptable for you (upstream)?

@mmistakes
Copy link
Owner

Sure looks like a good solution. Submit a PR and I'll test it out.

On Saturday, September 27, 2014, Roman Valls Guimerà <
[email protected]> wrote:

Thanks @mmistakes https://github.com/mmistakes, I went for this option:

brainstorm/brainstorm.github.io@cc10b7f
https://github.com/brainstorm/brainstorm.github.io/commit/cc10b7f7b3385

I think it's the one what gives most flexibility (and granularity). Would
you like me to prepare a pullrequest? Are the changes acceptable for you?


Reply to this email directly or view it on GitHub
#94 (comment)
.

mmistakes added a commit that referenced this issue Dec 27, 2014
Solves issue #94 with disqus comments being enabled by default globally
mmistakes added a commit that referenced this issue Dec 27, 2014
# By Roman Valls Guimera
# Via Michael Rose (1) and Roman Valls Guimera (1)
* 'master' of https://github.com/mmistakes/so-simple-theme:
  Solves issue #94 with disqus comments being enabled by default globally
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
Solves issue mmistakes#94 with disqus comments being enabled by default globally
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
# By Roman Valls Guimera
# Via Michael Rose (1) and Roman Valls Guimera (1)
* 'master' of https://github.com/mmistakes/so-simple-theme:
  Solves issue mmistakes#94 with disqus comments being enabled by default globally
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
Solves issue mmistakes#94 with disqus comments being enabled by default globally
robercano pushed a commit to robercano/robercano.github.io that referenced this issue Mar 30, 2016
# By Roman Valls Guimera
# Via Michael Rose (1) and Roman Valls Guimera (1)
* 'master' of https://github.com/mmistakes/so-simple-theme:
  Solves issue mmistakes#94 with disqus comments being enabled by default globally
brandon-jimenez pushed a commit to brandon-jimenez/resume that referenced this issue Mar 5, 2021
brandon-jimenez pushed a commit to brandon-jimenez/resume that referenced this issue Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants