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

Can't scan QR code from desktop in dark mode #2004

Closed
jagerman opened this issue Oct 30, 2021 · 3 comments
Closed

Can't scan QR code from desktop in dark mode #2004

jagerman opened this issue Oct 30, 2021 · 3 comments

Comments

@jagerman
Copy link
Member

If Session is in dark mode and I go to my profile -> QR code, my Android device refuses to find the QR code no matter how long I hover it over the code displayed on the screen. In light mode it finds it instantly.

I played around with the image of the code, and found out it has to do with the amount of whitespace padding around the code -- I found some references that a QR code requires a minimum of two squares of padding on all sides (apparently with 4 recommended), but the QR code we display in dark mode only has 1 square width of padding on each side.

Unscannable dark mode QR currently produced by Session desktop:
image

I edited this in GIMP to add padding; with an extra block-width of padding all around my Android phone picks it up instantly, like it does in light mode:
image

Bilb added a commit to Bilb/session-desktop that referenced this issue Nov 4, 2021
@peepo5
Copy link

peepo5 commented Nov 10, 2021

This may be a specific thing with the app you are using, but this seems like a good idea just so it does not trip out any more QR scanners. Normally there would not be a dark background around the edges so it may be thinking that it is part of the QR code.

@peepo5
Copy link

peepo5 commented Nov 10, 2021

I see this is fixed on the webrtc branch. 👍

@Bilb
Copy link
Collaborator

Bilb commented Dec 21, 2021

Yeah, should be fixed now

@Bilb Bilb closed this as completed Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants