-
Notifications
You must be signed in to change notification settings - Fork 12
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 favicon inversion (#114) #224
base: 0.13.0
Are you sure you want to change the base?
Conversation
I'm guessing I have merged this via e901efc and dda4587 and 7f8d4d3 It is live now in the dev version. Please test it and let me know if I missed something. |
Yes, and I improved more by receiving your comment. In this PR, I merged those commits with |
@Croydon I'm not sure why I was assigned 😅 What should I do? |
@mmktomato Nothing specifically. I'm just assigning always the person/people (mainly) responsible for the work. Private life keeps me busy right now but I will soon merge this into a beta version in the |
I see. Thanks!! |
It has been a minute. Sorry! Do you remember the details of what exactly was improved here? Is the threshold when it inverts or not better in this version? |
I've recreated this pull request.
original pull request: #222