Feature: Reply Counts

[epicsam123]

Productivity feature: include reply counts for both the "load more" and show/hide replies buttons.

Easily tested this through browser dev tools since all source code changes were purely JavaScript, so this PR can be expedited.

[unixfox]

hey @AHOHNMYC @ChunkyProgrammer, could you please take a look at this PR for the JS side? thank you :D

[ChunkyProgrammer]

Haven't tested but I have this one comment

[unixfox]

hello @ChunkyProgrammer, could do a final check in order to see if it is ok? thanks.

[ChunkyProgrammer]

@unixfox Code LGTM

It also LGTM on the video page and community page

Here are some screenshots:

Initial look:

after opening replies (top)

after opening replies (bottom)

after loading all the comments:

Comment count change:

before loading all:

after loading all:

Community page:

[Minecon724]

I don't see how the (x/x) is useful. I would keep "view x replies" loaded or not.

[epicsam123]

I don't see how the (x/x) is useful. I would keep "view x replies" loaded or not.

I personally like the prospect where with long comment chains you can know how far you have read into a comment's long thread of replies (characteristic of the first couple of comments due to popularity ranking) after initially collapsing the comment and then returning to the top of the comment section to find that same comment. I can modify the counter to only display the final count i.e. (40) vs (40/40) when all the comments are loaded.

[Minecon724]

I don't see how the (x/x) is useful. I would keep "view x replies" loaded or not.

I personally like the prospect where with long comment chains you can know how far you have read into a comment's long thread of replies (characteristic of the first couple of comments due to popularity ranking) after initially collapsing the comment and then returning to the top of the comment section to find that same comment. I can modify the counter to only display the final count i.e. (40) vs (40/40) when all the comments are loaded.

Maybe do that in a "title" attribute? It's displayed on hover
Either way I suggest making it clearer like "(40/40 loaded)"

[epicsam123]

I don't see how the (x/x) is useful. I would keep "view x replies" loaded or not.

I personally like the prospect where with long comment chains you can know how far you have read into a comment's long thread of replies (characteristic of the first couple of comments due to popularity ranking) after initially collapsing the comment and then returning to the top of the comment section to find that same comment. I can modify the counter to only display the final count i.e. (40) vs (40/40) when all the comments are loaded.

Maybe do that in a "title" attribute? It's displayed on hover Either way I suggest making it clearer like "(40/40 loaded)"

Added to both title and sub text (for mobile users)

[epicsam123]

Fractions are read as left to right in RTL languages:
https://en.wikipedia.org/wiki/Modern_Arabic_mathematical_notation