-
Notifications
You must be signed in to change notification settings - Fork 197
Conversation
1a099da
to
daf0e3c
Compare
(by sheer coincidence, I used that first "cat" result to test the mature filter in the dev environment, so that's why it only appears in the mature results) |
Two comments:
|
@panchovm the Settings thing is how it's described in the issue (See "Filter" in #848). The Filter By is a bug. Thanks for spotting it. I'll fix it |
9d36d04
to
388f341
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This image (top right, the red square is my own) comes up regardless of whether or not "Enable Mature Content" is selected, but I'm not sure if this is frontend or api side. I reported the image :)
Another thing is that the "Mature" text and the X in the filter block button don't look like they're aligned vertically. I think they are literally speaking, but they look slightly off in my opinion.
Side Note: In package.json, I noticed that the author is "sebworks", and the description is "Common Creative search", which doesn't look right to me.
Little bits here and there that I think could/should be addressed, but it looks good!
@akmadian the results come from the API. I think the filter will work better as more images are classified as nsfw. @aldenstpage can clear it up better for you As for the alignment, can you open an issue so that this can be done separately? |
@akmadian In this initial pass, we don't have any way to automatically identify an image as NSFW, we just created the means to flag an image manually and exclude them from the search results. In the future we'll use keywords to flag potential NSFW images. This ticket has some details: cc-archive/cccatalog-api#482 |
Fixes #435 by @kgodey
Developer Certificate of Origin
Developer Certificate of Origin