glutanimate / review-heatmap

Anki add-on to help you keep track of your review activity
https://ankiweb.net/shared/info/1771074083
Other
1.2k stars 89 forks source link

Remove inverted colors in night mode #114

Closed warrenalphonso closed 2 years ago

warrenalphonso commented 3 years ago

Description

Fixes #59

Night mode is still used, eg for keeping empty tiles black instead of white. But for color schemes, a 'small' number is assigned a light color while a 'large' number is assigned a darker color. There's no default inversion for night mode. I've left the CSS in, so maybe we can add a checkbox in Options to let the user choose Inverted colors if they'd like. (I tried doing this myself, but I kept getting a KeyError. I thought all I had to do was modify config_defaults.synced or config_defaults.profile, but neither worked. Would appreciate some help on this if it's deemed a good idea to add a checkbox in Options to let the user choose inverted colors.)

Checklist:

Please replace the space inside the brackets with an x and fill out the ellipses if the following items apply:

glutanimate commented 2 years ago

Hey Warren, I really appreciate the contribution (and my apologies that it took me ages to respond)!

I agree that making this configurable would be a good idea (with the current behavior kept for existing users and perhaps even new users as people have likely gotten used to it). Given that the add-on has undergone quite a bit of refactoring recently and with more on the way in terms of how styling works, I think it's best to close this PR for now. However, I've added this to my list of things to look into as soon as that refactoring work is done (and will be happy to ping you either once the actual feature is in, or once the code is in a state where someone could reasonably pick this option up again as a PR).

Either way, thanks again for the PR!