Skip to content

Conversation

@lucasew
Copy link
Contributor

@lucasew lucasew commented Jul 23, 2024

fix: encoder/terminal: swap symbol colors, readers read more easily when the foreground is the dark color

cc @qsantos

…hen the foreground is the dark color

Signed-off-by: lucasew <[email protected]>
@qsantos
Copy link
Owner

qsantos commented Jul 25, 2024

With the camera of the Pixel 4a, I can detect the QR code for Hello, World! when using master, but not when using lucasew:20240723-fix-colors. This is the same when using white-on-black and using black-on-white. Did you observe the behavior with a specific reader?

@lucasew
Copy link
Contributor Author

lucasew commented Jul 25, 2024

master:
image

this pr:

image

what is going on lol

all readers are having trouble scanning this

@qsantos
Copy link
Owner

qsantos commented Jul 25, 2024

Would it make sense to add some -i / --invert option with the current inverted colors, to cover all bases?

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

Successfully merging this pull request may close these issues.

2 participants