Add text to encourage comments on blog post; invisible by default

[stefanv]

No description provided.

[netlify]

✅ Deploy Preview for scientific-python-hugo-theme ready!

Name	Link
🔨 Latest commit	decc6bc
🔍 Latest deploy log	https://app.netlify.com/projects/scientific-python-hugo-theme/deploys/698bd983db8fd6705f391286
😎 Deploy Preview	https://deploy-preview-700--scientific-python-hugo-theme.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 94 Accessibility: 97 Best Practices: 100 SEO: 91 PWA: - View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[agriyakhetarpal]

I think this would be nice. Side note; the linkchecker plugin invites a lot of noise. I'd like to change that later on, as we've discussed previously. Do you think you'll change the visibility of the please-comment container using JS? Also, shouldn't we do this for the blog/single.html layout directly?

[stefanv]

In the accompanying PR on the blog site, I show how to enable it using CSS
I wasn't convinced that it is the best design, but it does the trick
I'm happy to have you make a recommendation!

[agriyakhetarpal]

Thanks! Hmm, no, CSS is indeed the best design. For some reason I thought https://github.com/scientific-python/blog.scientific-python.org/blob/a2e9beccc6d837d7bce1b89d97ec29cdd1e1cb33/layouts/partials/posts/comments.html was part of the theme and I am surprised it isn't (so my suggestion to put it in blog/single.html doesn't make sense right now). I think we should add that JS here and enable it behind a Hugo config flag (keeping it disabled by default).

Is this PR ready for review/merging as is? I'm happy to approve if it works for you.

[stefanv]

For the theme, I think having giscus as the default commenting mechanism makes most sense.

The setup we have for connecting to Discourse is very bespoke.

[stefanv]

I think this one's okay. Not perfect, but fine for now.

[agriyakhetarpal]

Let's land this, then. Thanks @stefanv!

I am temporarily retracting my approval for now, as I did not mean to approve this but rather only comment – see above

[stefanv]

I made the change you suggested, thanks @agriyakhetarpal

[stefanv]

(In general, btw, I am totally fine to have commits pushed to my branches, in case that helps streamline things.)

[agriyakhetarpal]

Noted! Thanks! Approving now, let's test this out soon-ish.