Skip to content

Preserve repeating background layers as native group backgrounds - #3555

Merged
chubes4 merged 1 commit into
trunkfrom
fix/issue-3552-blocks-engine
Oct 27, 2026
Merged

chubes4 merged 1 commit into
trunkfrom
fix/issue-3552-blocks-engine

Conversation

@chubes4

@chubes4 chubes4 commented Oct 27, 2026

Copy link
Copy Markdown
Contributor

Summary

Repeating background-only layers now stay backgrounds instead of becoming intrinsic-size images. WordPress receives explicit auto sizing when no size was authored, preserving the tile rather than stretching it.

What changed

  • Reuse CoverStyleResolver repetition detection and preserve resolved URL, repeat, size, position, and presentation attributes.
  • Replace the empty wrapper with the background group instead of nesting it.
  • Add regression coverage for default and authored sizes, shorthand repetition, and the non-repeating cover image path.
  • Rendered before/after evidence using WordPress and Chrome: before, one 40×40 image; after, intrinsic 40px tiles repeat across an 800×206 box. Heading line-height was pinned to reproduce the reported box height; screenshot pixels confirmed repetition.

Verification

  • passed on 16634ab. Base trunk@d1e6b64 unchanged.
set -e
cd php-transformer
(test -d vendor || composer install --no-progress --no-interaction)
( while sleep 60; do echo "gate heartbeat: composer test still running"; done ) &
hb=$!
trap 'kill $hb 2>/dev/null || true' EXIT
composer test
  • CI: pending
Cook run
  • Gate 1: tree f6b70d4ec965396a0b5efd7602fab5a865d9eab9, fingerprint d4b87f8cd818afa7699db6848eaa92b0b6db71847a5b54891c8bc555c6f19d46

AI assistance

Tool: Homeboy (pi) · Model: gpt-6.1-sol (tier roadie/cook-heavy) · Used for: AI implemented the transformer changes, added regression coverage, and verified suite and rendered behavior.

Closes #3552

@chubes4
chubes4 merged commit d08165d into trunk Oct 27, 2026
15 checks passed
@chubes4
chubes4 deleted the fix/issue-3552-blocks-engine branch October 27, 2026 01:39
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.

Repeating background layer converts to a single intrinsic-size core/image (tiling lost)

1 participant