Fix: Embed: Block transformed to a paragraph keeps classes #44947
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.
What?
Fix #39616. Remove internal css classes, added by the embed block when using some variations like youtube, when transforming the block to another type.
Why?
Related to #38964, the feature allow user-added custom CSS classes to be carried over when transforming blocks. Unfortunately, the Embed block stores its aspect ratio classes as custom CSS classes and they are also kept during transform.
How?
Reusing the same filter as the custom classname support
blocks.switchToBlockType.transformedBlock
, we apply theremoveAspectRatioClasses
to the list of classname when transforming an embed block to another type. This allow only remove the aspect ration classes and the keep user-added ones.Testing Instructions (Base on the instructions from the original issue)
wp-embed-aspect-16-9
andwp-has-aspect-ratio
wp-embed-aspect-16-9
andwp-has-aspect-ratio
anymore