jekyllDecent icon indicating copy to clipboard operation
jekyllDecent copied to clipboard

Add comment count to Disqus

Open jwillmer opened this issue 8 years ago • 7 comments

I thought about changing the "Load comments" into "Load 5 Comments" to give an indication how much comments are there. The goal is to do this without loading the whole Disqus overhead. Turns out, someone did it already and inspecting his demo (use raymondcamden and open dev tools) I can see that Disqus does not load any bloatware.

Let's implement it? @alexsimo

jwillmer avatar Jun 01 '17 09:06 jwillmer

Not bad idea! But would be the flow? an infinite scroll? or maybe the user will need to click the button every time wants 5 more comments?

alxsimo avatar Jun 01 '17 13:06 alxsimo

No, sorry, my example is misleading. I only like to show that are a number N amount of comments already. So I know if there are comments I can read.

Load Comment Section (5💬)

jwillmer avatar Jun 01 '17 21:06 jwillmer

Oh! Understood now! Yes! Looks good for me! I'll assign myself but not sure when can make it.

alxsimo avatar Jun 01 '17 21:06 alxsimo

Seems I can't do it myself. Assign me please

alxsimo avatar Jun 01 '17 21:06 alxsimo

Strange, I can't find you in the assignee drop-down. I wrote GitHub support to have a look. I will assign you as soon as I can :point_up:

jwillmer avatar Jun 02 '17 08:06 jwillmer

From the support team:

This has to do with the fact that the user opened that original issue and it can therefore be assigned to him, versus in this case, you opened the issue and the user is not a collaborator and thus can't be assigned to the issue.

I can promote you to collaborator if you don't mind to still use pull requests and not commit directly.

jwillmer avatar Jun 02 '17 11:06 jwillmer

Sure!

On Fri, Jun 2, 2017, 13:34 Jens Willmer [email protected] wrote:

From the support team:

This has to do with the fact that the user opened that original issue and it can therefore be assigned to him, versus in this case, you opened the issue and the user is not a collaborator and thus can't be assigned to the issue.

I will promote you to collaborator if you don't mind to still use pull requests and not commit directly.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/jwillmer/jekyllDecent/issues/40#issuecomment-305762083, or mute the thread https://github.com/notifications/unsubscribe-auth/AAjXiXuAvKk1X2TNYXMTFG3tcd1J1oF5ks5r__NIgaJpZM4Nssin .

--

Alexandru Vladut Simonescu

Software Craftsman. Android Developer. Yoga Lover.

Blog http://blog.alexsimo.com/ | GitHub https://github.com/alexsimo | LinkedIn https://es.linkedin.com/in/alexandrusimonescu | Twitter http://twitter.com/alexsimonescu

alxsimo avatar Jun 02 '17 11:06 alxsimo