site banner

Bugs, Suggestions, Small Comments, And Site News

Have you found a problem in the site? Do you want to make a suggestion on improvement? Do you just want to say "hi everyone"? Post it here!

If you'd like to help with development, check out the Github and the dev Discord. We have a practically infinite list of small things that need to be fixed or changed.

24
Jump in the discussion.

No email address required.

Might be easier to just store it in the browser's localStorage, though that has the issue of not being shared across multiple clients.

Might be, but it's still a significant amount of coding work. If we're doing that we may as well put it in the DB.

It's more performant to get the clients to store that information, and handle filtering posts, rather than your servers storing that N*M mapping and doing various joins. I don't care about sharing across devices, but I do care about it staying collapsed on one device! (and of course your server costs. If you implement it server-side, then it can only slow down your comment-loading queries even more).

Wouldn't a simple permalink-to-boolean mapping do the trick?

EDIT: Or just an array of permalinks actually

It's not a hard thing to describe, it's just a bunch of work to implement. Like, yeah, you need that! And now you need to change all the SQL queries used to build the page to ensure that they're efficient. And now you need to make sure the page works well. And now you need the API entry and you need to ensure your rate limiting is appropriate.

Is it doable? Absolutely! Is it more than a few days of work? Absolutely not! But meanwhile we have 64 issues and rising; maybe this just isn't the juiciest fruit available, y'know?

Oh yeah I was talking about a localStorage approach. Unless the show/hide logic is done server-side and hidden comments aren't loaded. If the logic is all client-side then depending on the architecture (and JS framework) it could be a few lines of code and a 15 minute job.