-
Notifications
You must be signed in to change notification settings - Fork 350
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
fix: puck header style for #168 #174
fix: puck header style for #168 #174
Conversation
@Yuddomack is attempting to deploy a commit to the Measured Team on Vercel. A member of the Team first needs to authorize it. |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Sorry for slow review but looks good @Yuddomack! One minor comment.
@@ -318,7 +318,7 @@ export function Puck({ | |||
display: "grid", | |||
padding: 16, | |||
gridTemplateAreas: '"left middle right"', | |||
gridTemplateColumns: "288px auto 288px", | |||
gridTemplateColumns: "288px auto 344px", |
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.
@Yuddomack thanks for updating, although you missed my other comment:
gridTemplateColumns: "288px auto 344px", | |
gridTemplateColumns: "344px auto 344px", |
b25468f
to
995b386
Compare
Merged, thanks @Yuddomack! |
I slightly modified the style of the header area for #168