-
Notifications
You must be signed in to change notification settings - Fork 30
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/new vaults table #340
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Running Lighthouse audit... |
f013e86
to
209fdfb
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.
Looks really good @peterslany :) - couple of comments/questions, but not necessarily change requests.
Thanks for the review @tomjeatt! I pushed some updates so feel free to re-review :) |
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 great :)
Adding
Table
andNewVaultsTable
component. This component displays information about current and upcoming vaults. The table is responsive: when screen is smaller than table, horizontal scrolling is activated. The columns of table are scaled to full width by default.Note: The CTA is invisible on Interlay since it doesn't have styling defined for Interlay secondary variant yet, once we define it the CTA will display styled without any needs to update the vaults table component.