Media: Graduate client-side media processing from experimental #75112
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…table Move the media processing module from lib/experimental/media/ to lib/media/ as part of graduating this feature from experimental status. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove the experiment toggle as this feature is now graduated to stable. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Remove the experiment check and always load the media processing module from its new stable location. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Replace experiment check with gutenberg_client_side_media_processing_enabled filter that defaults to true. This allows sites to disable the feature while keeping it enabled by default. Also rename the window global from __experimentalMediaProcessing to __clientSideMediaProcessing to reflect stable status. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add early return when client-side media processing is disabled via filter. Update docblocks and comments to reflect stable status. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Use the gutenberg_client_side_media_processing_enabled filter to determine whether to use the 6.9 compat REST controller for attachments. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Update all JavaScript references from __experimentalMediaProcessing to __clientSideMediaProcessing to reflect the stable status of this feature. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Add documentation to test files and add new tests for the gutenberg_client_side_media_processing_enabled filter. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
adamsilverstein
requested review from
ellatrix and
spacedmonkey
as code owners
github-actions
bot
added
[Package] Editor
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
Consolidates the duplicated `gutenberg_client_side_media_processing_enabled` filter calls into a single helper function `gutenberg_is_client_side_media_processing_enabled()` in lib/load.php. This follows the same pattern as `gutenberg_is_experiment_enabled()` and reduces code duplication across three files. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
adamsilverstein
added
[Type] Task
|
Size Change: +26 B (0%) Total Size: 3 MB
ℹ️ View Unchanged
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
lib/experimental/media/tolib/media/gutenberg_client_side_media_processing_enabledfilter (defaults totrue) allowing sites to disable the feature if neededwindow.__experimentalMediaProcessingtowindow.__clientSideMediaProcessingTest plan
add_filter( 'gutenberg_client_side_media_processing_enabled', '__return_false' );to a pluginCloses #75062
🤖 Generated with Claude Code