Download Statistics on "Frequently Downloaded" page

[JohannesSeikowsky]

using html tags to hide the Download statistics on the "Frequently Downloaded" page by default, but keeping them easily accessible on click right there.

Means the few people who are used to seeing the statistics there can still do so very easily, but the statistics don't get in the way of normal users who are primarily interested in finding books!

Technically this is a minimal change, only adding characters to the $s string. relatively low-risk I would say (which matters given the situation with the php scripts)

[JohannesSeikowsky]

@rtonsing do you think this would be ok to deploy? Or do we have some way of "dry-testing" this change before just to be a bit more certain it doesn't break anything (which seems unlikely to me, but better to be sure if possible)

[rtonsing]

Should be fine to merge to dev, & test at https://dev.gutenberg.org/.

[eshellman]

could you post images of what this change does? or perhaps the output html files? We currently don't have an easy way to pretest changes in the php files.

[JohannesSeikowsky]

[JohannesSeikowsky]

And if you click on that, it expands to look like the page looks now by default (obviously).

[JohannesSeikowsky]

@rtonsing I think you said before that this can be tested on dev. Is that so? If yes, we should do it.
Having the download statistics in the current place is an obvious UX mistake.

[rtonsing]

It is my understanding that merging to dev updates the dev site, but @eshellman would know for sure.

[JohannesSeikowsky]

I think we should risk deploying this. It fixes a very obvious UX mistake in a simple way. And in the pretty unlikely event that it should break something, we just roll back immediately (not ideal but not the end of the world)

[JohannesSeikowsky]

on a general level we should have proper testing for the php scripts. But that's not a small thing and imo we shouldn't be held back by that for simple change like this