Skip to content
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

Tom/layout components #331

Merged
merged 19 commits into from
Jun 14, 2022
Merged

Tom/layout components #331

merged 19 commits into from
Jun 14, 2022

Conversation

tomjeatt
Copy link
Collaborator

@tomjeatt tomjeatt commented Jun 10, 2022

This PR adds Grid, GridItem and Stack components. I've kept these light with minimal config, as we're likely to want the styles here to be quite restrictive to keep the design/UX consistent (i.e. no property for over-riding the grid gap).

@vercel
Copy link

vercel bot commented Jun 10, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
interbtc-ui-interlay ✅ Ready (Inspect) Visit Preview Jun 13, 2022 at 2:34PM (UTC)
interbtc-ui-interlay-testnet ✅ Ready (Inspect) Visit Preview Jun 13, 2022 at 2:34PM (UTC)
interbtc-ui-kintsugi ✅ Ready (Inspect) Visit Preview Jun 13, 2022 at 2:34PM (UTC)
interbtc-ui-kintsugi-testnet ✅ Ready (Inspect) Visit Preview Jun 13, 2022 at 2:34PM (UTC)
interbtc-ui-testnet ✅ Ready (Inspect) Visit Preview Jun 13, 2022 at 2:34PM (UTC)

@github-actions
Copy link

Running Lighthouse audit...

@tomjeatt tomjeatt marked this pull request as ready for review June 13, 2022 14:38
Copy link
Contributor

@peterslany peterslany left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me 👍

@tomjeatt tomjeatt merged commit 3070a31 into master Jun 14, 2022
@tomjeatt tomjeatt deleted the tom/layout-components branch June 14, 2022 09:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants