-
Notifications
You must be signed in to change notification settings - Fork 179
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: component theming #1791
base: soul/integrations/makeswift
Are you sure you want to change the base?
feat: component theming #1791
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
|
2bb617f
to
ab7bc1a
Compare
682d4d2
to
5b45156
Compare
5b45156
to
4b7d2bc
Compare
4b7d2bc
to
a1adf36
Compare
@@ -16,15 +14,13 @@ export default async function DefaultLayout({ params, children }: Props) { | |||
setRequestLocale(locale); | |||
|
|||
return ( | |||
<MakeswiftProvider previewMode={(await draftMode()).isEnabled}> |
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.
Moved <MakeswiftProvider>
to the root root layout so that theming can be applied to all pages.
<VercelComponents /> | ||
</body> | ||
</html> | ||
</MakeswiftProvider> |
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.
label: 'Sections', | ||
type: sections, | ||
}), | ||
}, |
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.
titleFont: Font({ label: 'Title font', variant: false }), | ||
contentFont: Font({ label: 'Content font', variant: false }), | ||
light: colorGroup('Light'), | ||
dark: colorGroup('Dark'), |
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.
background: Color({ label: 'Background' }), | ||
text: Color({ label: 'Text' }), | ||
focus: Color({ label: 'Focus' }), | ||
close: closeButton, |
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.
tertiary: colorGroup('Tertiary'), | ||
ghost: colorGroup('Ghost'), | ||
focus: Color({ label: 'Focus' }), | ||
}, |
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.
borderRadius: Number({ label: 'Border radius (px)', defaultValue: 16 }), | ||
focus: Color({ label: 'Focus' }), | ||
light: colorGroup('Light'), | ||
dark: colorGroup('Dark'), |
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.
a1adf36
to
02add2f
Compare
type: { | ||
focus: Color({ label: 'Focus' }), | ||
light: colorGroup('Light'), | ||
dark: colorGroup('Dark'), |
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.
font: Font({ label: 'Font', variant: false }), | ||
text: Color({ label: 'Text' }), | ||
focus: Color({ label: 'Focus' }), | ||
}, |
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.
searchResult: searchResultGroup('Search result'), | ||
cartCount: cartCountGroup('Cart count'), | ||
locale: localeGroup('Locale'), | ||
}, |
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.
layout: Shape.Layout.Popover, | ||
type: { | ||
light: colorGroup('Light'), | ||
dark: colorGroup('Dark'), |
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.
focus: Color({ label: 'Focus' }), | ||
light: colorGroup('Light'), | ||
dark: colorGroup('Dark'), | ||
}, |
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.
footerLinkHover: Color({ label: 'Footer link hover' }), | ||
footerCopyright: Color({ label: 'Footer copyright' }), | ||
focus: Color({ label: 'Focus' }), | ||
}, |
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.
playBorder: Color({ label: 'Play border' }), | ||
playBorderHover: Color({ label: 'Play border hover' }), | ||
playText: Color({ label: 'Play text' }), | ||
focus: Color({ label: 'Focus' }), |
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.
@@ -17,7 +17,7 @@ | |||
"@conform-to/react": "^1.2.2", | |||
"@conform-to/zod": "^1.2.2", | |||
"@icons-pack/react-simple-icons": "^10.2.0", | |||
"@makeswift/runtime": "0.0.0-snapshot-20241217195602", | |||
"@makeswift/runtime": "0.0.0-snapshot-20241220223738", |
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.
New runtime
snapshot w/ Shape
v2 support.
02add2f
to
ea30743
Compare
label: 'Section', | ||
layout: Shape.Layout.Popover, | ||
type: { | ||
maxWidth: widthGroup('Max width'), |
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.
434647d
to
5ef9a8c
Compare
ea30743
to
5d71056
Compare
export const Component = async ({ | ||
snapshotId, | ||
...props | ||
}: { |
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.
Userland RSC for MakeswiftComponent
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-latest-qd5n7nxvo-bigcommerce-platform.vercel.app 🖥️ DesktopWe ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:
📱 MobileWe ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:
|
What/Why?
Visually editable site and component themes
Testing
Kapture.2024-12-23.at.19.10.53.mp4