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

Improve content warning explanation (fixes #220) #403

Closed
wants to merge 2 commits into from

Conversation

Keltena
Copy link
Contributor

@Keltena Keltena commented Aug 4, 2024

This pull request updates the explanation of content warnings on the entry registration form by:

  1. Using the clearer, more succinct wording suggested in Content warning submission text #220:

    We do ask that you use this field only for real and useful content warnings. Humor and irony (e.g. "Contains trace amounts of peanuts") can be put in the blurb instead.

  2. Commenting out the line "Players will have the option to hide content warnings from their ballots", as that feature hasn't actually been implemented yet. (See /ballot should have show/hide content-warning controls #173.)

Keltena added 2 commits August 4, 2024 08:34
Until we add an option to hide content warnings (see iftechfoundation#176), this is misleading.
@coveralls
Copy link

Coverage Status

coverage: 65.299%. remained the same
when pulling 2c572a3 on Keltena:cw-explanation
into b60f63b on iftechfoundation:main.

@Keltena Keltena closed this Aug 5, 2024
@Keltena Keltena deleted the cw-explanation branch August 5, 2024 00:43
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