Skip to content

refactor(php-transformer): move empty inline geometry lowering into Elements/EmptyInlineGeometryLowering - #5276

Open
chubes4 wants to merge 3 commits into
trunkfrom
simplify/empty-inline-geometry
Open

chubes4 wants to merge 3 commits into
trunkfrom
simplify/empty-inline-geometry

Conversation

@chubes4

@chubes4 chubes4 commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Part of #4368 (slice 2z). Stacked on #5275, which must merge first.

What

  • New owner, Elements/EmptyInlineGeometryLowering. Moves emptyInlineGeometryBlock and its six helpers verbatim out of HtmlCompilation. They lower an empty inline tree whose source CSS still generates a line box or flex track into the existing layout shell.
  • Mechanical substitutions only:
    • SourceDom::attr.
    • preservedSourceControlBlock calls $this->blocks->htmlPreservationBlock() through the injected SourceBlockCreator, the same method it reached through $this.

HtmlCompilation: 7037 → 6914 lines.

Verification


AI disclosure: Claude Opus (Anthropic) via Pi/Roadie moved the code with a mechanical mover script, ran the Lab gate and A/B, and wrote this PR for Chris Huber.

…ts/InertElements

Moves isInertRuntimeMediaPlaceholder, isInertHiddenSvgStorage and
hiddenSvgStoreHasNonSvgConsumer out of HtmlCompilation verbatim.
…lements/EmptyInlineGeometryLowering

Moves emptyInlineGeometryBlock and its six helpers out of HtmlCompilation
verbatim. preservedSourceControlBlock reaches htmlPreservationBlock
through the injected SourceBlockCreator.
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.

1 participant