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

Gray border with transparent png: from 5.2.0 #253

Open
wclr opened this issue Dec 20, 2018 · 13 comments
Open

Gray border with transparent png: from 5.2.0 #253

wclr opened this issue Dec 20, 2018 · 13 comments

Comments

@wclr
Copy link

wclr commented Dec 20, 2018

I have original image:
circle

From 5.2.0 the result image has gray border:
android-chrome-144x144

In early version there is no border.

There also was an issue before:
#123

@alexander-akait
Copy link
Member

@whitecolor feel free to send a PR with fix

@wclr
Copy link
Author

wclr commented Dec 20, 2018

I think it is quite a severe regression btw. For now I prefer just to downgrade https://github.com/brunocodutra/webapp-webpack-plugin

@brunocodutra

@brunocodutra
Copy link
Collaborator

@whitecolor any idea where the border is coming from?

@wclr
Copy link
Author

wclr commented Jan 1, 2019

Well, should be some stuff related to picture generation? Maybe svg2png? Need to see what changed between favicons 5.1.1 and 5.2.0.

@brunocodutra
Copy link
Collaborator

Right, it was certainly due to some dependency being updated, in which case this issue needs to be reported upstream once we find the culprit.

@egemon
Copy link

egemon commented Mar 21, 2019

Sorry guys, thank u for doing such a good stuff. Do u think we can see a fix in some time?

@alexander-akait
Copy link
Member

PR welcome, it is open source and many peoples don't have enough time

@brunocodutra
Copy link
Collaborator

brunocodutra commented Mar 21, 2019

Actually a recent PR #254 replaced svg2png by sharp.
@egemon do you have the means to test favicons@master and verify that the issue is gone?

@mikroware
Copy link

So that PR only changed the handling over svg images? I have this gray border issue with png images. So that PR would not have fixed my case. I will try to test this soon though and also export my image to svg to see if that works better.

@brunocodutra
Copy link
Collaborator

brunocodutra commented Mar 23, 2019

Good point, I don't see why we use both Jimp and Sharp, we should move to Sharp everywhere.

@kghost Do you think you can help us out?

@nirui
Copy link

nirui commented Sep 28, 2019

The same problem brought me here.

And actually, I found that when .svg file is used instead of the .png one, the problem is gone.

I don't actually know any detail on the issue, but maybe take a look at the parameter used during PNG file processing?

@wclr
Copy link
Author

wclr commented Jul 9, 2020

@niruix thanks for pointing to svg solution.

@papiliond
Copy link

I still have this issue.

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

No branches or pull requests

7 participants