Skip to content

Fix storefront wrapper theme inheritance - #1492

Merged
wcole1-godaddy merged 1 commit into
godaddy:mainfrom
sbolinger-godaddy:fix/storefront-theme-inheritance
Oct 2, 2026
Merged

wcole1-godaddy merged 1 commit into
godaddy:mainfrom
sbolinger-godaddy:fix/storefront-theme-inheritance

Conversation

@sbolinger-godaddy

Copy link
Copy Markdown
Contributor

Summary

Storefront wrappers force white backgrounds and dark text on dark websites. Default .commerce-storefront to a transparent background and inherited text color, while preserving explicit --commerce-surface and --commerce-text overrides.

Update the compiled CSS assertions and document the new defaults. Controls and the portalled cart drawer retain their existing palette and can be customized through the theme variables.

Changeset

  • Changeset added for a patch release of @godaddy/gd-commerce-storefront.

Test Plan

  • pnpm --filter @godaddy/gd-commerce-storefront build — passed.
  • pnpm --filter @godaddy/gd-commerce-storefront test — all 28 tests passed, including compiled CSS and Tailwind v3 compatibility checks.
  • pnpm --filter @godaddy/gd-commerce-storefront lint — passed.
  • git diff --check — passed.

@sbolinger-godaddy
sbolinger-godaddy requested a review from a team as a code owner October 2, 2026 16:29
@changeset-bot

changeset-bot Bot commented Oct 2, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: e876027

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@godaddy/gd-commerce-storefront Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@wcole1-godaddy
wcole1-godaddy merged commit b9f3415 into godaddy:main Oct 2, 2026
3 checks passed
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.

2 participants