-
Notifications
You must be signed in to change notification settings - Fork 2
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
Design #62
Design #62
Changes from all commits
943c5a5
5e582e1
b5b0ad7
4c9b817
be944e9
68c8ce9
811af94
10f3a63
4a28133
7dcc1a0
3757ad6
67856d6
3716e5c
59845df
675f223
09c2890
ce8ba8c
a589531
688ef41
3442b02
9182fa6
58d1339
b04acd9
47c98a9
89557df
b836169
e7325ec
c644cf1
2986006
ea803bb
1420e04
2592a69
9398506
2d1611e
f43ffb2
6d12f47
8235e0f
85d677c
d171c23
376d3c7
9e5e12d
966fb2a
0c8d9c8
cea2c28
1cf4420
44c9011
2344f8c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
nodeLinker: node-modules |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -140,7 +140,9 @@ function AddNetworkDrawer({ isOpen, onClose }: Props): JSX.Element { | |
} | ||
> | ||
{!!apiError && ( | ||
<FormHelperText color="red">{apiError}</FormHelperText> | ||
<FormHelperText textColor="brand.red"> | ||
{apiError} | ||
</FormHelperText> | ||
)} | ||
</FormInput> | ||
<FormInput | ||
|
@@ -209,10 +211,10 @@ function AddNetworkDrawer({ isOpen, onClose }: Props): JSX.Element { | |
</DrawerBody> | ||
|
||
<DrawerFooter> | ||
<Button variant="outline" mr={3} onClick={close}> | ||
<Button variant="whiteModal" mr={3} onClick={close}> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I don't like that all primary buttons and secondary ones (like Cancel / Back) now looks the same... |
||
Cancel | ||
</Button> | ||
<Button type="submit" colorScheme="green" onClick={onSubmit}> | ||
<Button type="submit" variant="whiteModal" onClick={onSubmit}> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. And if it is a kinda standard style for the button (except some specific cases like buttons in the create wallet flow) — then mb let's make it default and don't put in every button? |
||
Save | ||
</Button> | ||
</DrawerFooter> | ||
|
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.
Let's use named sizes instead of pixels, e.g.: