Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sensitive content analysis #1689

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Sensitive content analysis #1689

wants to merge 6 commits into from

Conversation

bryanmontz
Copy link
Contributor

@bryanmontz bryanmontz commented Nov 14, 2024

Issues covered

I'm not sure if there's a ticket for this.

Description

Adds sensitive content analysis, Apple's built-in system for detecting nudity in images.

How to test

First make sure that Sensitive Content Warning is enabled at the OS level. Go to Settings.app -> Privacy & Security -> Sensitive Content Warning.
image

The real analyzer will be used in this state. If you want to see how the UI feels when content is flagged but without actually exposing yourself to sensitive content, go to the Settings view in Nos and override the analyzer to always flag either or both downloaded and uploaded images.
image

Screenshots/Video

When images are flagged as sensitive or you have overridden the analyzer in Settings:
image

When you try to upload an image that is flagged or you have overridden the analyzer in Settings:
image

Copy link
Contributor

@joshuatbrown joshuatbrown left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overall this looks good and seems to work well. I did notice an issue where I was unable to open the image viewer when I tapped an image in a gallery (this may be true for single images in a post, too). I wonder if something here might be overriding the tap recognizer for images?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants