Block Editor: Enqueue script modules via enqueue_block_assets action#10597
Block Editor: Enqueue script modules via enqueue_block_assets action#10597sirreal wants to merge 9 commits into
Conversation
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
| 'scripts' => $scripts, | ||
| 'styles' => $styles, | ||
| 'scripts' => $scripts, | ||
| 'script_modules' => $script_modules, |
There was a problem hiding this comment.
Script modules print the script tags, but also print HTML for the @wordpress/a11y module if it's enqueued. That probably needs to be injected into the editor as well?
wordpress-develop/src/wp-includes/class-wp-script-modules.php
Lines 937 to 951 in 63b3a8f
There was a problem hiding this comment.
Since print_a11y_script_module_html runs at wp_footer, yes, it seems this will need to be captured separately above with more output buffering.
The very selective nature of picking and choosing what gets captured to be injected into the editor iframe, however, gives me pause about how well the editor will replicate the frontend in which themes and plugins could be adding other arbitrary markup to wp_head and wp_footer.
There was a problem hiding this comment.
Good point. A specific action for the editor iframe analogous to wp_head to capture arbitrary HTML for inclusion may be best here. Then this would just run the action instead if calling select methods.
Co-authored-by: Weston Ruter <[email protected]>
This value is set and accessed in the printing flow and should be reset
The editor package prints styles and scripts sequentially. Provide HTML as styles to ensure its printed in legacy editor packages.
17136e4 to
28305f9
Compare
Trac ticket: https://core.trac.wordpress.org/ticket/64360
Requires Gutenberg changes to the block editor. See WordPress/gutenberg#73765.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.