Related to T2814. QRCode has been given a margin-top of 15%. If you think it would look better, I would stack this diff with one which moves the new InfoCtn above the QRCode so that all content is visible above the fold.
Details
Details
- Reviewers
bytesofman - Group Reviewers
Restricted Project - Commits
- rABC0fe80ccc4d0b: [Cashtab] Improve vertical centering of QRCode in Receive.js
cd web/cashtab && npm start
navigate to Receive screen
observe that the QRCode component is pushed further towards the center of the screen
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- improve-vertical-centering-receive
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 21639 Build 42918: Build Diff cashtab-tests Build 42917: arc lint + arc unit
Event Timeline
Comment Actions
No need to stack. The info thing on the QR code screen is temporary and will be removed soon.