react-native-qrcode-generator icon indicating copy to clipboard operation
react-native-qrcode-generator copied to clipboard

Remove this fork and rewrite a new library for this.

Open rishichawda opened this issue 5 years ago • 6 comments

This code is shit. Remove everything, rewrite and publish as v1.1 under the react-native-qrcode-generator package name.

rishichawda avatar Apr 11 '19 07:04 rishichawda

@rishichawda Let me know if you'd like help with that :)

gitstud avatar Mar 10 '20 07:03 gitstud

@gitstud I would appreciate it very much but I am not getting time myself to work on this. I will notify you as soon as I start! Fingers crossed for this weekend. 🤞

rishichawda avatar Mar 12 '20 17:03 rishichawda

I would recommend the React Native QRCode SVG, as it renders native Views without the need for workarounds with canvas and webview.

douglasjunior avatar Aug 22 '20 14:08 douglasjunior

Thanks for the recommendation. I have used that one before. Part of the reason why I had forked and maintained this was because I needed to use it in one of the projects and needed to generate a plain QRCode, nothing fancy. react-native-qrcode-svg library comes with other functionalities which I didn't need. That means it comes with the additional code that I didn't need. Hence, this.

rishichawda avatar Aug 22 '20 20:08 rishichawda

SVG overhead is much less than the version using webview.

The react-native-qrcode-svg just use the qrcode to generate the paths and then converts the paths to use the react-native-svg, it is a very lean implementation, there is no extra code or workarounds like in webview version that uses canvas and web others web technologies.

Think, with webview you need to load an entire browser just to display a QR code image.

douglasjunior avatar Aug 23 '20 01:08 douglasjunior

As I have said earlier, this was forked for a project where I needed a simple QRCode view. That project already had a WebView. So this was the best option for that project. I'm aware of the internal implementation of the library you have mentioned and would undoubtedly use for other projects where it seems fit. I'm also mindful of how badly this library is doing this. Hence this issue was created.

rishichawda avatar Aug 23 '20 07:08 rishichawda