favicons icon indicating copy to clipboard operation
favicons copied to clipboard

Gray border with transparent png: from 5.2.0

Open wclr opened this issue 6 years ago • 13 comments

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: https://github.com/itgalaxy/favicons/issues/123

wclr avatar Dec 20 '18 17:12 wclr

@whitecolor feel free to send a PR with fix

alexander-akait avatar Dec 20 '18 17:12 alexander-akait

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

@brunocodutra

wclr avatar Dec 20 '18 18:12 wclr

@whitecolor any idea where the border is coming from?

brunocodutra avatar Dec 31 '18 17:12 brunocodutra

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.

wclr avatar Jan 01 '19 16:01 wclr

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.

brunocodutra avatar Jan 01 '19 16:01 brunocodutra

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

egemon avatar Mar 21 '19 11:03 egemon

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

alexander-akait avatar Mar 21 '19 11:03 alexander-akait

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?

brunocodutra avatar Mar 21 '19 20:03 brunocodutra

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.

mikroware avatar Mar 22 '19 14:03 mikroware

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?

brunocodutra avatar Mar 23 '19 11:03 brunocodutra

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?

nirui avatar Sep 28 '19 03:09 nirui

@niruix thanks for pointing to svg solution.

wclr avatar Jul 09 '20 16:07 wclr

I still have this issue.

papiliond avatar Nov 18 '20 15:11 papiliond