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 favicon inversion (#114) #224

Open
wants to merge 1 commit into
base: 0.13.0
Choose a base branch
from

Conversation

mmktomato
Copy link
Contributor

I've recreated this pull request.
original pull request: #222

@Croydon
Copy link
Owner

Croydon commented Feb 24, 2019

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.

@mmktomato
Copy link
Contributor Author

Yes, and I improved more by receiving your comment.
#222 (comment)
#222 (comment)

In this PR, I merged those commits with --squash option.

@mmktomato
Copy link
Contributor Author

@Croydon I'm not sure why I was assigned 😅 What should I do?

@Croydon
Copy link
Owner

Croydon commented Mar 5, 2019

@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 0.13.0 branch 😄

@mmktomato
Copy link
Contributor Author

I see. Thanks!!

@Croydon
Copy link
Owner

Croydon commented Jul 28, 2023

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?

@Croydon Croydon modified the milestones: 0.14.0, 0.15.0 Aug 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants