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

Different QR-Code in the preview #50

Closed
DrNion opened this issue Oct 31, 2021 · 2 comments
Closed

Different QR-Code in the preview #50

DrNion opened this issue Oct 31, 2021 · 2 comments
Labels
enhancement New feature or request

Comments

@DrNion
Copy link

DrNion commented Oct 31, 2021

The informations of both QRCodes stay identical though. Just the pixels are different.

To be honest I don't know if this creates a problem for other QRCode scanner. Just wanted to mention it fyi.

@michaeltroger michaeltroger added the enhancement New feature or request label Nov 4, 2021
@michaeltroger
Copy link
Owner

michaeltroger commented Nov 4, 2021

This is working as designed. I also mention this in the app description.
So for the functionality of a QR code reader the visual appearance doesn't matter and I would call it a nice to have for the human eye only.

If you wish a more technical explanation:
The same QR code text content can be rendered with different error correction levels.
The library I'm using for reading QR codes (ZXing) doesn't seem to be capable of telling me the error correction of the original barcode (happy for any suggestions how to achieve that). Therefore I'm always using the same error correction, no matter how the original barcode was rendered.

If something is still unclear then feel free to reopen this

@DrNion
Copy link
Author

DrNion commented Nov 6, 2021

Thanks for the explanation, there is no need to reopen it then :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants