-
Notifications
You must be signed in to change notification settings - Fork 41
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
Update AlertBox colors #1439
Update AlertBox colors #1439
Conversation
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
Deploying with Cloudflare Pages
|
Codecov Report
@@ Coverage Diff @@
## master #1439 +/- ##
==========================================
- Coverage 83.02% 82.92% -0.10%
==========================================
Files 154 154
Lines 4029 4042 +13
Branches 726 728 +2
==========================================
+ Hits 3345 3352 +7
- Misses 684 690 +6
Flags with carried forward coverage won't be shown. Click here to find out more.
|
0120652
to
a20a107
Compare
a20a107
to
5655b17
Compare
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.
Is there an issue with bg colors or am I looking at wrong Figma?
}, | ||
'alert-box-error-background': { | ||
dark: '#ffe7d9', | ||
light: '#f26111', |
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.
Can we align text color as well? otherwise light/error will fail accessibility color contrast with current #444
. Mocks are using diff font color.
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 do that in a separate PR. I think the font color change applies everywhere
src/styles/theme/ThemeProvider.tsx
Outdated
light: '#f3d45e', | ||
}, | ||
'alert-box-error': { | ||
dark: '#d24c00', |
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.
#F26111
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.
oops, seems to have changed
}, | ||
'alert-box-warning': { | ||
dark: '#f3d45e', | ||
light: '#f0e8cb', |
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.
#F3D45E
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.
I think that's a bug on mocks - border #F3D45E
should match the icon #f0e8cb
This reverts commit 3a0faf7 and re-implements it so multiline alerts like /create-wallet on a phone are not stretched out.
1a86bf7
to
a31fc60
Compare
https://www.figma.com/file/G6EVdJhfGbDBZRdpWnUyC7/Wallet-Design-System?type=design&node-id=22-1429&t=E44uYOZqmDSuC2xp-0