-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
Explorer transfer table1 #3101
Explorer transfer table1 #3101
Conversation
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.
looks mostly good, i feel like this bundles together a few parts which should come separately.
We could really use this table view as a re-usable component for other kinds of data. I would like it if we can break that out of the part of this PR that relies on mock data we don't have ready yet - would make it easier to review.
Remove component with mock data from the home page and added the old pagination. Pending design update. |
categoryName: 'transactions' as Category, | ||
isLink: true, | ||
isCopyButton: false, | ||
alttext: 'More Transaction', |
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.
alttext: 'More Transaction', | |
alttext: 'More Transactions', |
@stella3d thanks |
Screen.Recording.2022-07-07.at.5.48.15.PM.mov