Skip to content

feat(blocks): add Brand Kit logo variation - #104

Merged
TheAggressive merged 2 commits into
masterfrom
feat/issue-96-brand-logo
Sep 21, 2026
Merged

TheAggressive merged 2 commits into
masterfrom
feat/issue-96-brand-logo

Conversation

@TheAggressive

Copy link
Copy Markdown
Owner

Summary

  • classify Logo as a constrained WordPress Core Image variation backed by one read-only Brand Kit binding source
  • version the Brand Kit asset snapshot and import an HTTPS raster Site Logo with alt text, intrinsic dimensions, and an optional homepage link
  • resolve the frozen logo deterministically through the existing image renderer with bounded proportional dimensions
  • cover editor serialization, compiler failures, client structure, persistence, and package output

Classification

Logo uses core/image plus campaignbridge/brand-data. CampaignBridge does not register a duplicate Logo block or a second asset store. Theme import copies the current Site Logo into Brand Kit version 3, and compilation never reads live theme or Media Library data.

Validation

  • pnpm build
  • pnpm qa:fast
  • pnpm test — 1,132 tests, 5,305 assertions
  • pnpm exec playwright test tests/e2e/brand-logo.spec.ts
  • pnpm release:package && pnpm release:verify

Refs #96

@github-actions github-actions Bot added area:blocks WordPress email blocks area:email Email compiler or campaign workflow area:frontend Frontend source or Node tooling area:php PHP or Composer code area:tests Automated tests or test configuration risk:high Security or release sensitive; never auto-merged type:feature New capability needs-attention Automation stopped; maintainer action required labels Sep 21, 2026
@TheAggressive
TheAggressive merged commit 0b21f73 into master Sep 21, 2026
21 checks passed
@TheAggressive
TheAggressive deleted the feat/issue-96-brand-logo branch September 21, 2026 12:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:blocks WordPress email blocks area:email Email compiler or campaign workflow area:frontend Frontend source or Node tooling area:php PHP or Composer code area:tests Automated tests or test configuration needs-attention Automation stopped; maintainer action required risk:high Security or release sensitive; never auto-merged type:feature New capability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant