Skip to content

Restore Webmention forms on block themes and hide classic-only checkbox - #672

Merged
pfefferle merged 2 commits into
mainfrom
fix/block-theme-form-setting
Oct 5, 2026
Merged

pfefferle merged 2 commits into
mainfrom
fix/block-theme-form-setting

Conversation

@pfefferle

@pfefferle pfefferle commented Oct 5, 2026 •

Copy link
Copy Markdown
Owner

Release archives excluded build/form/block.json through the /build/*/block.json export-ignore rule. Without that metadata, WordPress cannot register or automatically insert the Webmention Form block, while block themes skip the classic form. This leaves no Webmention form even when the checkbox is enabled. On notiz.blog, the metadata URL returns 404 while the form JavaScript returns 200.

Include the form block metadata in release archives and only skip the classic form when the form block is registered. Other build-entry metadata stays excluded.

Hide the "Show a Webmention form" checkbox only on block themes and ignore its saved value when using Block Hooks. Classic themes retain the checkbox and its behavior; the custom help-text field remains. The form block can be moved or removed in the Site Editor.

Validation:

  • 13 form tests pass, including actual hook insertion and rendering with a block theme and the legacy setting disabled, plus the classic fallback when block registration is missing.
  • git archive HEAD build/form includes build/form/block.json.
  • PHP coding checks and git diff --check pass.
  • PHPUnit reports the existing configuration warning about the source element under PHPUnit 9.

Follow-up to #667.

@pfefferle pfefferle changed the title Ignore classic form setting for block themes Restore Webmention forms on block themes and hide classic-only checkbox Oct 5, 2026
@pfefferle
pfefferle merged commit d0b7749 into main Oct 5, 2026
8 checks passed
@pfefferle
pfefferle deleted the fix/block-theme-form-setting branch October 5, 2026 14:30
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