Remove files that are already included in WordPress Core #7727
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.
Resolves https://github.com/Automattic/sensei-security/issues/17. (Cherry-picked from https://github.com/Automattic/sensei-security/pull/28.)
Proposed Changes
core-pattern
block that was needed for WP 5.8. I checked WP 5.9 and confirmed that the block is registered there.assets/dist/css/jquery-ui.js
file after webpack creates it. As we are enqueuing thejquery-ui.css
file in PHP, rather than importing it via a JS file, Webpack generates a correspondingjquery-ui.js
file that is essentially empty. That is true of all CSS files that we handle this way, and is a well-documented problem. I tried using this Webpack plugin but it didn't work, so instead opted for a quick and dirty solution and just delete this particular file after Webpack does its thing.Testing Instructions
npm run build
.assets/dist/css/jquery-ui.js
file.Pre-Merge Checklist