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

implement zen mode #544

Open
wants to merge 29 commits into
base: master
Choose a base branch
from
Open

Conversation

glbert-does
Copy link
Member

@glbert-does glbert-does commented Dec 22, 2024

closes #541

this pull request is based on the django_update pr #527.

it implements zen mode using template tags. i think there are likely good arguments to do it using css instead, but i don't think the code would have been much simpler, since there is currently nothing that marks something as a rating consistently, so one would still have to add a lot of html attributes. possibly using css would be more future proof, but then again, there is nothing preventing anyone not using a new html attribute for newly added ratings.
that is why i decided to hide ratings completely using tags.

i still think the main work here is to find all places that reference ratings. this should be mostly done, but of course it is plausible that i missed some places.

i decided to not remove ratings from the stats pages. they are almost completely anonymous there. but maybe one should? i prefer to wait for zen mode users to complain about it.

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.

hide all ratings on the site
1 participant