Skip to content
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

something is wrong in this library. #49

Closed
justraman opened this issue May 10, 2021 · 7 comments
Closed

something is wrong in this library. #49

justraman opened this issue May 10, 2021 · 7 comments
Labels
reviewed Temporary label to review all open and closed PR

Comments

@justraman
Copy link
Contributor

this lib is dependent on https://kazuhikoarase.github.io/qrcode-generator/js/demo/.

when I create a QR there it's scannable from all devices and all QR apps.

but when I create the same QR from https://qr-code-styling.com/ without(image and styling) it's not scannable from less popular QR apps.

how this is possible?

reference app for android: https://play.google.com/store/apps/details/?id=com.mrakash.qrcodescanner
ios: Facebook app, QR scanner Pro

@randomprogramming
Copy link

Don't think it's this library, I've had the same issues using pretty much any QR Code styling library. As soon as you change the styling of a QR code even the slightest a lot of web and android based qr code scanners just don't want to scan the code.

Interestingly, iOS never has any issues from my testing.

@DewiarQR
Copy link

DewiarQR commented Aug 20, 2021

Столкнулся с той же проблемой. Даже самый простой, совсем не измененный QR код в этой библиотеке - не сканируется. При этом в библиотеке https://kazuhikoarase.github.io/qrcode-generator/js/demo/ он сканируется очень хорошо.

В БИБЛИОТЕКЕ ЕСТЬ КРИТИЧЕСКАЯ ОШИБКА! Если ее не исправить, примерно половина распространенных сканеров не смогут расшифровать эти QR-коды. Ошибка в том, что пиксели-точки развернуты не верно, они отображены зеркально и повернуты на 90 градусов, это не правильно.

Screenshot_2

@DewiarQR
Copy link

На изображении сверху 2 QR кода с одной и той же зашифрованной информацией. Слева, корректный и правильный QR-код, считывающийся всеми сканерами. Справа - QR-Code styling - с ошибкой размещения точек (окрашены бордовым цветом). Видно, что они развернуты не верно.
Я пытался в библиотеку внести правку, перед отображением этот контейнер отображаю зеркально и сдвигаю на 180 градусов, это частично решает проблему. Но проблема всеравно дает о себе знать при загрузке фото в центр, фотография также переворачивается зеркально.

dmitrystas added a commit to dmitrystas/qr-code-styling that referenced this issue Feb 15, 2022
@dmitrystas dmitrystas mentioned this issue Feb 15, 2022
@dmitrystas
Copy link
Contributor

#110

@HenkVanMaanen
Copy link
Contributor

@dmitrystas this can be fixed too by the following code, it needs to be fixed in both QRSVG.ts and QRCanvas.ts:

WRONG:
dot.draw( xBeginning + i * dotSize, yBeginning + j * dotSize,

FIX:
dot.draw( yBeginning + j * dotSize, xBeginning + i * dotSize,

@HenkVanMaanen
Copy link
Contributor

See fix here: https://github.com/sallandpioneers/qr-code-styling

This fork will be maintained until further notice by Salland Pioneers (business).

@kozakdenys
Copy link
Owner

Hi @dmitrystas @justraman @HenkVanMaanen @randomprogramming @DewiarQR , sorry for leaving you for so long

The fix was released in version v1.7.1 and above.

@kozakdenys kozakdenys added the reviewed Temporary label to review all open and closed PR label Oct 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewed Temporary label to review all open and closed PR
Projects
None yet
Development

No branches or pull requests

6 participants