Fixing duplicated wp-block classes in block-list #24466
Merged
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.
Description
Note: This is my first PR.
Fixes issue #24367
I removed the isAlign check for block-wrapper.js of block-list in block-editor. This check is already done in block.js. Doing it again causes the wp-block class to show up twice in the block-list.
How has this been tested?
I tested it on Firefox in Ubuntu to see if anything was broken.
I added some blocks in the editor. I couldn't find anything that was misaligned or didn't show up. All the blocks in the block-list had one
wp-block
class. No more duplicatedwp-block
classes.Then, I ran the
npm run test
script. I didn't get any errors.Types of changes
Checklist: