-
Notifications
You must be signed in to change notification settings - Fork 10
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
Refactor styling of debug dialog #2802
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Current dependencies on/for this PR:
This comment was auto-generated by Graphite. |
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.
d675c78
to
7cb46bc
Compare
7217a76
to
5a28eb5
Compare
7cb46bc
to
498e9de
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.
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, this layout take much less markup!
@@ -47,7 +47,7 @@ export const DebugResumeSessionSection = () => { | |||
|
|||
const Layout = styled.form({ |
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: now we have grid with single line, right? Maybe flexbox would be more obvious
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.
Hmmm but then we need to add "flex" properties to children to determine how they scale horizontally
That's why I like grid -- it let's you determine layout from the parent 🙂
@robinandeer started a stack merge that includes this pull request via Graphite. |
5a28eb5
to
e8ab425
Compare
Graphite rebased this pull request as part of a merge. |
498e9de
to
22f768d
Compare
@robinandeer merged this pull request with Graphite. |
Describe your changes
Justify why they are needed
I wanted it go along with input fields
The button height doesn't match input fields so they look a little strange
Checklist before requesting a review