-
Notifications
You must be signed in to change notification settings - Fork 477
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 creation design + step 1 #967
Conversation
Deploying with Cloudflare Pages
|
ESLint Summary View Full Report
Report generated by eslint-plus-action |
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.
Creating custom components just to add a couple of lines of CSS to MUI seems...excessive?
I would just make one Card module, and either pass props for the header and content or export them as named exports.
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.
Since we're building a new file structure anyway, I suggest we don't use the old "open" folder and put stuff under src/components/new-safe
(to be used by both Create and Add Safe).
.back { | ||
margin: var(--space-4) 0; | ||
} | ||
|
||
.title { | ||
padding-bottom: var(--space-2); | ||
} |
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.
These one-liners could be my={4}
and pb={2}
respectively.
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.
Button
does not accept shorthands so I've added an sx
to it.
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.
Nice!
What it solves
Resolves #961
How this PR fixes it
This implements the new creation/load flow layout, as well as the first step of creation. The progress bar is a placeholder as it will rely on all steps/progress and the stepper hook.
Note: the info widget is yet to be included as it has not been QA'd.
How to test it
Open
/demo
and observe the new design.Screenshots