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

Users with "Always Show CW Field" option enabled cannot control whether or not their images are marked as sensitive. #64

Closed
jebutton opened this issue Jun 1, 2019 · 2 comments
Assignees
Labels
instaging The fix for this is in staging, ready to be tested for prod

Comments

@jebutton
Copy link
Collaborator

jebutton commented Jun 1, 2019

Expected behavior

When a user has the "Always Show CW Field" option enabled should be able to control whether or not the images they post are marked as sensitive or not. So, if a user with that option enabled wants to post an image without marking it as sensitive, they should be able to deselect the checkbox that marks the image as sensitive before they post it.

Actual behavior

Users with the "Always Show CW Field" option are unable to deselect the checkbox to mark the image as not sensitive. This forces them to always post images as sensitive, regardless if that was their intent or not.

Steps to reproduce the problem

  1. Either Create a New Account or go into the preferences page and enable the option to "Always Show CW Field" under the "Mods" section.

Specifications

I found this issue in the staging version of BeachCity, which was "beachcity1.7.0".

I tested this issue on both Google Chrome (on a Windows 10 Pro install with most recent updates) and on an Ubuntu 18.04 VM with Firefox 67.0.

@jebutton
Copy link
Collaborator Author

jebutton commented Jun 1, 2019

BeachCity_mastodon Issue #64 Screenshot 01 Here is a screenshot of what the relevant section of the window looks like where a user has the CW button disabled and is trying to post an image. The "Mark Media as Sensitive" button can't be deselected.

@aarchiba
Copy link

aarchiba commented Jun 1, 2019

Note that the situation is exactly the same as when the user has the CW button enabled and has pressed it.

A solution should permit three possibilities: post has a CW and image is marked sensitive, post has no CW but image is marked sensitive, and post has no CW and image is not marked sensitive.

Be aware that this may interact with the transformation of a post with a CW but no body apart from an image into a post whose body contains the CW plus image (marked sensitive) and that has no CW. Particularly we want to make sure that such images don't fail to get marked sensitive.

@robinmaypanpan robinmaypanpan self-assigned this Jun 2, 2019
@robinmaypanpan robinmaypanpan added the instaging The fix for this is in staging, ready to be tested for prod label Jun 2, 2019
robinmaypanpan pushed a commit that referenced this issue Jun 4, 2019
Fixes #64 Set sensitivity of uploaded content when always show spoile…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
instaging The fix for this is in staging, ready to be tested for prod
Projects
None yet
Development

No branches or pull requests

3 participants