libreddit icon indicating copy to clipboard operation
libreddit copied to clipboard

Published 20 hours ago • verified publisher icon libreddit

Implement "posts hidden because of NSFW"

Open sigaloid opened this issue 3 years ago • 3 comments
trafficstars

Fixes #159

sigaloid avatar May 12 '22 23:05 sigaloid

I like the idea of this but it seems to show up in pretty much ever feed under this current implementation, including r/popular: image

It might be best to curtail this to only NSFW subreddits where no content is viewable without enabling NSFW.

spikecodes avatar May 15 '22 23:05 spikecodes

Makes sense! I'll make the change so that it becomes "all" posts hidden by NSFW rather than any.

Unfortunately I can't find the API response that determines whether a sub is NSFW as a whole or if just a few posts are. Nonetheless the situation where a sub has a few non-NSFW posts and the rest would be hidden is probably an edge case (if a sub is marked NSFW every post is IIRC).

sigaloid avatar May 16 '22 10:05 sigaloid

Fixed logic to cover only when all posts are NSFW and user has disabled NSFW.

sigaloid avatar May 16 '22 20:05 sigaloid

@sigaloid: I'm going to close this in favor of the code you submitted to Ferrit, which is now available in a separate branch all-posts-hidden-nsfw. I'll open up a PR for that shortly.

Daniel-Valentine avatar Nov 07 '22 00:11 Daniel-Valentine