-
Notifications
You must be signed in to change notification settings - Fork 55
Add hide_qr method & clear_background param to show_qr #32
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thinking about this more, I want to change the clear_background parameter to hide_background to be consistent with the hide_QR style. I’m going to change it once I get in front of my computer again.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! One nitpicky thing then it should be ready to go.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Thank you @seantibor!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops, I was too quick. Lint is unhappy. Let me know if you have questions about it.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Thank you!
Updating https://github.com/adafruit/Adafruit_CircuitPython_LPS35HW to 1.1.2 from 1.1.0: > pylint fix > text fixes, product ID update Updating https://github.com/adafruit/Adafruit_CircuitPython_PyPortal to 3.0.2 from 3.0.1: > Merge pull request adafruit/Adafruit_CircuitPython_PyPortal#32 from seantibor/master Updating https://github.com/adafruit/Adafruit_CircuitPython_STMPE610 to 1.1.3 from 1.1.2: > Merge pull request adafruit/Adafruit_CircuitPython_STMPE610#8 from jpecor/patch-1 Updating https://github.com/adafruit/Adafruit_CircuitPython_Bitmap_Font to 1.0.3 from 1.0.2: > Merge pull request adafruit/Adafruit_CircuitPython_Bitmap_Font#11 from tannewt/improve_preload
I added a method that hides the QR code after it's shown. I also added a new parameter to the show_QR method that determines whether the background should be cleared before the QR code is added.