-
Notifications
You must be signed in to change notification settings - Fork 408
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
feat: default token list toggle #1549
Conversation
other changes: - Improved assets loading state
Branch preview✅ Deploy successful! https://spam_tokens_default_list--webcore.review-web-core.5afe.dev |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
ESLint Summary View Full Report
Report generated by eslint-plus-action |
@@ -19,6 +19,43 @@ import useBalances from '@/hooks/useBalances' | |||
import useHiddenTokens from '@/hooks/useHiddenTokens' | |||
import { useHideAssets } from './useHideAssets' | |||
|
|||
const skeletonCell = { |
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.
Nit: can we add the type for this to be safe.
<Skeleton width={'80px'} /> | ||
</Typography> | ||
</div> | ||
), | ||
}, | ||
balance: { | ||
rawValue: '0', | ||
content: ( | ||
<Typography> | ||
<Skeleton width={'32px'} /> | ||
</Typography> | ||
), | ||
}, | ||
value: { | ||
rawValue: '0', | ||
content: ( | ||
<Typography> | ||
<Skeleton width={'32px'} /> |
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.
<Skeleton width={'80px'} /> | |
</Typography> | |
</div> | |
), | |
}, | |
balance: { | |
rawValue: '0', | |
content: ( | |
<Typography> | |
<Skeleton width={'32px'} /> | |
</Typography> | |
), | |
}, | |
value: { | |
rawValue: '0', | |
content: ( | |
<Typography> | |
<Skeleton width={'32px'} /> | |
<Skeleton width="80px" /> | |
</Typography> | |
</div> | |
), | |
}, | |
balance: { | |
rawValue: '0', | |
content: ( | |
<Typography> | |
<Skeleton width="32px" /> | |
</Typography> | |
), | |
}, | |
value: { | |
rawValue: '0', | |
content: ( | |
<Typography> | |
<Skeleton width="32px" /> |
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.
LGTM
ESLint Summary View Full Report
Report generated by eslint-plus-action |
- link to actual help article
|
@liliya-soroka as discussed:
|
What it solves
Resolves #865
Todo
How this PR fixes it
tbd
How to test it
On staging CGW and görli / mainnet:
Analytics changes
tbd
Screenshots
tbd