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

update: Add info about the new clean up feature in ios #2833

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

friadev
Copy link
Contributor

@friadev friadev commented Dec 13, 2024

List of changes proposed in this PR:

  • added info about the new Clean Up feature in iOS that can automatically blur faces and redact information

@friadev friadev changed the title update: add info about the new clean up feature in ios update: Add info about the new clean up feature in ios Dec 13, 2024
Copy link

github-actions bot commented Dec 13, 2024

Your preview is ready!

Name Link
🔨 Latest commit 9f9ffa0
😎 Preview https://pr2833.unreviewed.privacyguides.dev/en/

Please note that this preview was built from an untrusted source, so it was not granted access to all mkdocs-material features. Maintainers should ensure this PR has been reviewed locally with a full build before merging.

@friadev friadev marked this pull request as ready for review December 13, 2024 09:26
@redoomed1 redoomed1 added the c:os operating systems and related topics label Dec 15, 2024
Comment on lines +264 to 273
<div class="admonition warning" markdown>
<p class="admonition-title">Warning</p>

Don't use Clean Up to remove text, block text out with a fully black box instead.

</div>

Otherwise, you can add a shape to fully block out a face or information.

- Open the **Photos** app and tap the photo you have selected for redaction
Copy link
Member

@redoomed1 redoomed1 Jan 6, 2025

Choose a reason for hiding this comment

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

Suggested change
<div class="admonition warning" markdown>
<p class="admonition-title">Warning</p>
Don't use Clean Up to remove text, block text out with a fully black box instead.
</div>
Otherwise, you can add a shape to fully block out a face or information.
- Open the **Photos** app and tap the photo you have selected for redaction
Our warning [against blurring text](../data-redaction.md) also applies here, so we recommend to instead add a black shape with 100% opacity over it. In addition to redacting text, you can also black out any face or object using the **Photos** app.
- Tap the image you have selected for redaction

This suggestion attempts to remove this admonition while retaining the same information. It also clarifies what a "fully black box" means.

@redoomed1
Copy link
Member

redoomed1 commented Jan 6, 2025

I also suggest renaming this section of the page to something like "Redacting Elements in Images" since the Clean Up feature now adds a redaction option that is not limited to blacking out faces/information that the current title indicates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c:os operating systems and related topics
Projects
Status: Has Approval
Development

Successfully merging this pull request may close these issues.

4 participants