endroid / qr-code

QR Code Generator
https://endroid.nl
MIT License
4.4k stars 725 forks source link

[Feature Request] Add Logo in EPS Output #148

Open fajinfo opened 6 years ago

fajinfo commented 6 years ago

Hello,

Would it be possible to add the support of a logo with the writers "Eps" or "Svg"?

I want to make a PDF with "SwissQR Invoice" and the print quality with the PNG is by far not perfect.

With the svg we get a perfect quality, but we can not add a logo.

Is this planned?

Fabian

endroid commented 6 years ago

Hi @swissfire this is not planned, but I agree this would be a nice feature. There is another issue open regarding the image quality so it looks like there is something wrong there.

endroid commented 6 years ago

FYI there was indeed an image quality issue caused by rounding block sizes. It is solved in 3.2.0, so you should be able to create pixel sharp PNG files now.

endroid commented 3 years ago

SVG Logo is possible so I changed the title.

libertey commented 2 years ago

Hey first of all, pretty good tool works pretty fine and smooth. But i have a little question. About the .eps files are you still working on the implementations of the logos in the eps? On the Tool im working right now version 3.4.0 is installed just for my knowledge if i simply have to update the tool or if i have to find an workaround to implement logos in an eps.

Greeting Paul

endroid commented 2 years ago

Hi @libertey currently I have no intention (because of time, priorities) on adding logo support in the EPS writer so unfortunately you would have to look for a workaround. If however you find something useful you can always let me know. Good luck!

libertey commented 2 years ago

Hi endroid, just in case you would to knwo i found an workaround in combination with imagick. There you dont have the exactly paths so its just one picture but the qr is scalable at least. so i render an picture with your package in png format and calculate then again with imagick to render it as an eps. Maybe it is useful if you wanna sometimes include the possibility for eps. Cheers

diealtebremse commented 8 months ago

@libertey can you provide the code of your workaround?