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

Hide pictures from the member picture page #1887

Open
BHenkemans opened this issue Aug 10, 2024 · 5 comments
Open

Hide pictures from the member picture page #1887

BHenkemans opened this issue Aug 10, 2024 · 5 comments
Labels
For: Backend Module: Decision For anything related to the Decision module Module: Photo For anything related to the Photo module Priority: Low Type: Idea Type: Privacy

Comments

@BHenkemans
Copy link
Contributor

BHenkemans commented Aug 10, 2024

What would you like?

I want the ability to hide pictures from the member picture page. This allows members to hide pictures (they do not like) from their profile page. This would not hide the picture from the original album. The picture would be greyed out on your page.

As an added bonus, we could also configure that if a picture is part of this list, it cannot become the picture of the week (which could prevent the ABC or board from having to take down a picture one of the members was already not fond of). (See this comment)

Why is this needed?

It gives a member some control over what pictures their profile shows when they are looked up by other members via the Find a Member feature

Other information

No response

@tomudding
Copy link
Member

Closely related to GH-1410. Potentially part of GH-148.

However, I do want to note that "As an added bonus, we could also configure that if a picture is part of this list, that it cannot become the picture of the week." is likely infeasible due to the fact there is no way we can ensure that the person tagged is actually in the picture and it is prone to abuse (there are plenty of pictures in which people are tagged that are not actually present in the picture).

@tomudding tomudding added Priority: Low For: Backend Module: Decision For anything related to the Decision module Module: Photo For anything related to the Photo module Type: Privacy labels Aug 10, 2024
@tomudding
Copy link
Member

tomudding commented Aug 10, 2024

For future reference, the flag has to be on the photo and not the tag (to prevent people from re-applying the tag if they notice a specific photo is hidden).

There will likely be three levels: hide all, hide selected, and hide none.

@BHenkemans
Copy link
Contributor Author

BHenkemans commented Aug 10, 2024

GH-1410 is slightly different but can be implemented while implementing this feature (as part of a ``picture privacy feature pack''). This issue reads as wholly opting out of tagging. This issue could be a middle ground. Allowing members to hide tags could remove one of the objections to being tagged, which is that their member picture page is filled with undesirable pictures.

I agree with your abuse comment; perhaps it is best to implement GH-1806 for that use case instead.

@tomudding
Copy link
Member

Allowing members to hide tags could remove one of the objections to being tagged, which is that their member picture page is filled with undesirable pictures.

I agree, and for more future reference, the photos should probably removed from the profile page and not blurred. This removes complexity from having to deal with the photo viewer and will ensure that people are less likely to go and search for which photo is "hidden".

@tomudding
Copy link
Member

Last for future reference (hopefully): when viewing your own profile, the images are shown but with an overlay (fa-eye-slash) if they are hidden.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
For: Backend Module: Decision For anything related to the Decision module Module: Photo For anything related to the Photo module Priority: Low Type: Idea Type: Privacy
Projects
None yet
Development

No branches or pull requests

2 participants