Page MenuHomePhabricator
Differential D14055

[web][native] add white frame around QR code
AcceptedPublic
Actions

Authored by kamil on Wed, Nov 27, 3:32 AM.
Tags
None
Referenced Files
F3393709: D14055.diff
Sat, Nov 30, 3:51 PM
Unknown Object (File)
Wed, Nov 27, 3:31 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
Unknown Object (File)
Wed, Nov 27, 3:30 PM
View All 17 Files
Subscribers
ashoat

Details

Reviewers
angelika
tomek
Summary

ENG-9875.

Native
before:

IMG_4667.PNG (1×828 px, 81 KB)

after:
IMG_4669.PNG (1×828 px, 83 KB)

Web
before:

Screenshot 2024-11-27 at 12.07.01.png (1×1 px, 135 KB)

after:
Screenshot 2024-11-27 at 12.06.51.png (1×1 px, 131 KB)

Test Plan

On Android physical device:

  1. Scanning QR code when using Add from Linked devices works
  2. Scanning QR code using an app camera (deep linking) works

Diff Detail

Repository
rCOMM Comm
Branch
backup
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

kamil created this revision.Wed, Nov 27, 3:32 AM
kamil held this revision as a draft.
Herald added a subscriber: ashoat. · View Herald TranscriptWed, Nov 27, 3:32 AM
kamil edited the summary of this revision. (Show Details)Wed, Nov 27, 3:34 AM
kamil edited the test plan for this revision. (Show Details)
Harbormaster completed remote builds in B32881: Diff 46096.Wed, Nov 27, 3:49 AM
kamil published this revision for review.Wed, Nov 27, 3:52 AM
tomek accepted this revision.Wed, Nov 27, 7:28 AM
tomek added inline comments.
web/account/qr-code-login.css
52

We should avoid using the colors directly and instead use something from our color theme. A comment from web/theme.css explains the reasoning.

This revision is now accepted and ready to land.Wed, Nov 27, 7:28 AM

Revision Contents
Changeset List

PathSize
native/
qr-code/
8 lines
web/
account/
8 lines
4 lines

Diff 46096

View Options

native/qr-code/qr-code-screen.react.js

Loading...
View Options

web/account/qr-code-login.css

Loading...
View Options

web/account/qr-code-login.react.js

Loading...