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

feat: visually editable PDP #1808

Draft
wants to merge 130 commits into
base: soul/integrations/makeswift
Choose a base branch
from

Conversation

agurtovoy
Copy link

@agurtovoy agurtovoy commented Dec 25, 2024

What/Why?

Make product detail page visually editable.

Testing

Kapture.2024-12-25.at.21.53.44.mp4

jorgemoya and others added 30 commits December 12, 2024 16:47
* chore(core): use next canary and enable ppr

* fix: use suggested functions over deprecated ones

* chore(core): update tailwind config and layout to use Soul config

* fix: eslint issues

* chore(core): add roboto mono font

* chore(core): add @/vibes/* path in tsconfig

* feat(core): add FeaturedProductsList component and use in Homepage (#1673)

* feat(core): add FeaturedProductsCarousel and use in Homepage (#1675)

* feat(core): add best selling products to Hompage (#1676)

* chore: bump components to include latest Streamable util (#1681)

* chore(core): use next canary and enable ppr

* feat: replace catalyst homepage slideshow

* refactor: updated slideshow structure based on PR feedback

* feat: add translations for homepage slider

* fix: slide not in view when element inside is focused

---------

Co-authored-by: Jorge Moya <[email protected]>
@agurtovoy agurtovoy force-pushed the aleksey/vib-1043-component-theming branch 3 times, most recently from 624ef9d to 0fd8e93 Compare December 31, 2024 07:49
Base automatically changed from aleksey/vib-1043-component-theming to soul/integrations/makeswift December 31, 2024 17:10
This commits also updates:
- Make the Shape prop required. Makeswift supports this by default.
- Add default values to height and weight that matches the Soul default values.
- Move the banner id prop to be after allowClose.
- Rename and move some variables.
Copy link
Contributor

github-actions bot commented Jan 8, 2025

⚡️🏠 Lighthouse report

Lighthouse ran against https://catalyst-latest-n12omngfl-bigcommerce-platform.vercel.app

🖥️ Desktop

We ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:

Category Score
🟢 Performance 96
🟢 Accessibility 94
🟢 Best practices 100
🟠 SEO 82

📱 Mobile

We ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:

Category Score
🟠 Performance 79
🟢 Accessibility 94
🟢 Best practices 100
🟠 SEO 85

@migueloller migueloller force-pushed the soul/integrations/makeswift branch from 826b01e to b2f5a9a Compare January 8, 2025 03:24
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.

9 participants