HomePhabricator
Diffusion Comm cb62ca92a1c5

[native] add danger option to `PrimaryButton`

Tags
None
Referenced Files
F3449364: IMG_4790 (1).PNG
Tue, Dec 17, 7:06 AM
File Not Attached
F3449360: image.png
Tue, Dec 17, 7:06 AM
File Not Attached
F3449358: image.png
Tue, Dec 17, 7:06 AM
File Not Attached
Subscribers
None

Description

[native] add danger option to PrimaryButton

Summary:
ENG-9959.

According to this we want a red Remove button so adding this config here.

I was trying to follow the approach we had in the codebase and was looking for patterns, but perhaps I broke some rules in how we should use colors, so sorry for that in advance.

Dark:

image.png (160×746 px, 15 KB)

Light:

image.png (150×736 px, 16 KB)

Example screen where this is used:

IMG_4790 (1).PNG (1×828 px, 57 KB)

Depends on D14149

Test Plan: Render button using dark and light theme

Reviewers: ashoat, tomek

Reviewed By: ashoat

Differential Revision: https://phab.comm.dev/D14150

Details

Provenance
kamilAuthored on Thu, Dec 12, 11:03 AM
Reviewer
ashoat
Differential Revision
D14150: [native] add danger option to `PrimaryButton`
Parents
rCOMMc50663f4ab07: [native] implement screen after secondary device is connected
Branches
Unknown
Tags
Unknown