instant.io icon indicating copy to clipboard operation
instant.io copied to clipboard

Show QR code

Open ghost opened this issue 9 years ago • 17 comments

Show a QR code for the different links so that mobile devices can become peers without users having to type in the URL.

ghost avatar Dec 17 '15 22:12 ghost

I can give this a go if it's wanted.

joshterrill avatar Dec 30 '15 22:12 joshterrill

I only use instant.io occasionally, so don't feel obliged to do this for my sake. I do think that this feature would be very handy though.

ghost avatar Dec 31 '15 16:12 ghost

@joshterrill Yeah, that would be cool to try. But are there any good QR reading libraries in pure javascript? I've seen tons of QR code generators, but only a few that can take grainy images from a webcam and parse the QR codes out of them.

feross avatar Jan 01 '16 19:01 feross

I'll look around and post what I find before I really start digging into it.

joshterrill avatar Jan 02 '16 10:01 joshterrill

@feross I found just https://github.com/LazarSoft/jsqrcode, do you know any other qr reading in js?

xxxorg avatar Mar 02 '16 17:03 xxxorg

Would you accept PR just for the qr code generation (not the reading)?

xxxorg avatar Mar 02 '16 17:03 xxxorg

@xxxorg I don't think we need it to read... I think it's just QR code generation so that a mobile app who has a reader can scan it. Am I right?

joshterrill avatar Mar 11 '16 23:03 joshterrill

I would accept a PR just for generation for now. We can figure out reading later.

On Fri, 11 Mar 2016 at 15:18 Joshua Terrill [email protected] wrote:

@xxxorg https://github.com/xxxorg I don't think we need it to read... I think it's just QR code generation so that a mobile app who has a reader can scan it. Am I right?

— Reply to this email directly or view it on GitHub https://github.com/feross/instant.io/issues/94#issuecomment-195599483.

feross avatar Mar 12 '16 00:03 feross

I can recommend http://neocotic.com/qr.js/ for QR generation which I have been using for years in a web application.

nodiscc avatar Mar 12 '16 06:03 nodiscc

I just wrote this yesterday but forgot to commit it. I'll do it later today.

joshterrill avatar Mar 16 '16 08:03 joshterrill

As for qrcode generation there is also https://larsjung.de/kjua/. It works without jQuery and is very powerful. (alternatively this works with jQuery)

rugk avatar Aug 27 '16 20:08 rugk

@feross can you look into this ?

I know a dev like you would be pretty busy, but this feature would solve the issue of sharying a file with a phone easy peasy.

aswinmohanme avatar Jun 15 '18 16:06 aswinmohanme

I apologize for letting this just stay open like this. I've been pretty swamped at work. If the suggestion @rugk has looks good to everyone I can try to implement it over the 4th of July break this week. @feross since that is a different library and the other pull request is pretty old, should we close the other one out and start a new one? Or should I pull the existing one down and edit it?

joshterrill avatar Jul 04 '18 00:07 joshterrill

@feross I just finished writing the code... not sure how you want me to go about pushing it though. Should I pull down my other PR and modify that? Or use a new one since I'm using a different library than before?

joshterrill avatar Jul 04 '18 00:07 joshterrill

There were enough differences in the existing code base and what was in that old PR that I figured I could just create a new PR. If this isn't what you want to do, just let me know. I've created https://github.com/webtorrent/instant.io/pull/176 using the library @rugk suggested.

joshterrill avatar Jul 04 '18 01:07 joshterrill

Any update regarding the implementation status of QR codes? Feature request was created 5 years ago and pull request 1.5 years ago, so it's a bit sad that this wasn't added, yet.

ThelloD avatar Nov 29 '20 09:11 ThelloD

I would also really appreciate this feature. https://webwormhole.io/ is fantastic in terms of this; if one wants to, for example, transfer a file from their desktop to their phone, this would be greatly appreciated.

pojntfx avatar Dec 03 '20 10:12 pojntfx