-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
remove color inversion #14894
remove color inversion #14894
Conversation
If we do this, it will need a big callout in the release notes and a documentation update that inverts the explanations on how to remove the night mode colorization with an explanation on how to re-do it with CSS (for people that do not want change) https://github.com/ankidroid/Anki-Android/wiki/FAQ#advanced-formatting-tips Basically, this feature is relatively polarizing as an idea (most don't care, some hate it, some probably like it) and everyone seems to have in general learned to live with it. Changing it requires some care for that reason as it is likely to be disruptive for a lot of people |
Below are my general thoughts. I'd accept a PR which added a preference, defaulting to true for existing users, and false for new users A follow up would be to mark "true" as deprecated I don't like the feature (this breaks the canonical deck for a language I'm working with) We should aim to be compatible with Anki desktop, but we should do this slowly to avoid frustrating users I believe the steps would be something similar to:
|
The layers of filter for frustrated people would be the people that:
So I don't think that the number of affected people will be big and actually expect more people being frustrated because the color is inverted than because it is not inverted. Therefore, I prefer being disruptive and doing it at once (but I won't quote Zuckerberg motto here). The cautious approach is fine too, but it's up to the people that want to carry it given the limited human resources we have and the amount of stuff that is more important than this |
I think disrupt and "fix" with altered documentation / changelog mention is okay |
Disagree & commit: let's do this! I'm going to be working on deprecation in preferences for |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also LGTM, pending CI
Hi there @RobozinhoD! This is the OpenCollective Notice for PRs merged from 2023-12-01 through 2023-12-31 If you are interested in compensation for this work, the process with details is here: https://github.com/ankidroid/Anki-Android/wiki/OpenCollective-Payment-Process#how-to-get-paid We only post one comment per person per month to avoid spamming you, regardless of the number of PRs merged, but this note applies to all PRs merged for this month Please understand that our monthly budget is never guaranteed to cover all claims - the cap on payments-per-person may be lower, but we try to make our process as fair and transparent as possible, we just need your understanding. Thanks! |
Purpose / Description
Night mode wasn't consistent
Fixes
Approach
Remove night color inversion
How Has This Been Tested?
opened cards in night mode and the colors weren't inverted
Checklist
Please, go through these checks before submitting the PR.