Skip to content

Conversation

@dcalhoun
Copy link
Member

@dcalhoun dcalhoun commented Nov 3, 2025

What?

Remove the remote editor variant as a follow up to #209.

Why?

The remote editor variant is no longer used or necessary.

How?

Remove unused code. Rename and reorganize remaining code for clarity.

Testing Instructions

Smoke test the editor.

Accessibility Testing Instructions

N/A, no user-facing changes.

Screenshots or screencast

N/A, no visual changes.

@dcalhoun dcalhoun added the [Type] Code Quality Issues or PRs that relate to code quality label Nov 3, 2025
@dcalhoun dcalhoun force-pushed the task/remove-remote-editor-variant branch from c5fc89c to eacb4e1 Compare November 3, 2025 17:58
preferenceDispatch.set( 'core', 'editorMode', 'visual' );

registerCoreBlocks();
unregisterDisallowedBlocks( allowedBlockTypes );
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is now managed within src/utils/editor-environment.js after plugin assets are fetched. The response from the Jetpack REST API endpoint includes an allowed_block_types parameter.

Base automatically changed from feat/bundled-editor-supports-plugins to trunk November 6, 2025 22:19
@dcalhoun dcalhoun force-pushed the task/remove-remote-editor-variant branch from eacb4e1 to 6271be8 Compare November 7, 2025 00:30
The default editor now supports plugins, so the variant is not longer
necessary.
With the removal of the remote editor variant, the "bundled" term no
longer communicates intent. The utility manages setting up the editor
environment with a specific sequence.
This is now handled in the `editor-environment` logic as it depends upon
the plugins assets fetching.
@dcalhoun dcalhoun force-pushed the task/remove-remote-editor-variant branch from 6271be8 to 064717a Compare November 9, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants