-
Notifications
You must be signed in to change notification settings - Fork 127
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
QR code scanner/reader #52
Comments
Features may be:
|
This add-on shows what I mean: https://addons.mozilla.org/de/firefox/addon/qrcode-generator-and-reader/ |
Possibly a good lib: https://github.com/zxing-js/library |
@rugk Saw this after your link from #213
Just my 2 cents. |
This would definitively only be requested at runtime when you use it. (It would use WebRTC, which has this built-in.) An add-on actually does not have more permissions than a usual web page in this regards… |
Any one working on this issue??? |
If no one I want to work on this using html5-qrcode |
You can do so, that sounds great, but keep in mind this may be a big task and require some back- and forth. First of, you would have to:
|
The thing is html5 is very good to use as it is supported by so many browsers and I also think there are more better libs let me go through that!! |
Hey we just need a qr code scanner from the webcam right! then we can make it by Html5 with qr code lib. Which will let us use the webcam from the user to scan a qr. Let me know if this is what we need |
Technically it should work with the webcam and yes likely use a lib (which uses the WebRTC technologies for accessing a webcam, I guess)… But as indicated, one first has to think about a UI and find a good lib. Maybe also look for (preferably open-source) websites that already implement such a feature (and look what they use). |
Yeah I will make one which will have a decent UI and It can use WebRTC |
I'd love that to bridge phone texts / URLs to PC / laptop in some scenarios. 👯 |
There are also many JS libraries for just scanning a QR code:
So maybe we could integrate that, too.
The text was updated successfully, but these errors were encountered: