-
-
Notifications
You must be signed in to change notification settings - Fork 3.9k
Conversation
* Create QR page, with comment fix * Update on routes Co-authored-by: Eddie Jaoude <eddie@jaoudestudios.com>
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.
It's great having you contribute to this project
Welcome to the community 🤓If you would like to continue contributing to open source and would like to do it with an awesome inclusive community, you should join our Discord chat and our GitHub Organisation - we help and encourage each other to contribute to open source little and often 🤓 . Any questions let us know.
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.
LGTM
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.
Some thoughts.
- i think the "jumping" icon is a little bit distracting
- I would use a modal / popup for the QR code
- How about a toggle in the user.json to hide the QR code?
Thanks Tom for the feedback!
probably, this could be improved in another PR
I am not keen on modals for mobile
I think all profiles should have this available for quick access when at events etc |
Fixes Issue
fixes #2187
Changes proposed
Check List (Check all the applicable boxes)
Screenshots
Hidden
Show
Note to reviewers