-
Notifications
You must be signed in to change notification settings - Fork 3.2k
WIP: Update npm packages to latest versions for 6.4 Beta 1 #5262
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from 1 commit
Commits
Show all changes
77 commits
Select commit
Hold shift + click to select a range
f4144d8
Add changes from #5195
mikachan d1960cc
Add changes from #5235
mikachan 63217cd
Output of sync-gutenberg-packages so far
mikachan 59dd68d
Revert comments in update-core.php
mikachan 413751d
Update from latest sync script run
mikachan 3adb980
Potential fix for sync sourcemap warning
mikachan bb55583
Always enqueue prod Interactivity API
mikachan 22bad59
Update from successful sync packages run
mikachan dd177b0
Clean interactivity assets
mikachan f881350
Update from successful sync packages run
mikachan bac2694
Update src/wp-includes/script-loader.php
mikachan f06bde5
Merge branch 'trunk' into packpage-updates-6.4-beta1
mikachan ae84988
Results of sync packages script after merge with trunk
mikachan 670ad3a
Update packages using wp-6.4 dist-tag
mikachan b2a22e2
Merge branch 'trunk' into packpage-updates-6.4-beta1
mikachan 47dde56
Results of npm run build:dev
mikachan 836b9d0
Update tools/webpack/blocks.js
mikachan 9beee04
Re-run npm run build:dev
mikachan c22d71b
Remove sync pkg from sourcemap config
mikachan ea5bb8b
Update search/view path
mikachan 709b604
Update src/wp-includes/script-loader.php
mikachan 3cea4be
Docs: Correct typo in new `@since` tag.
desrosj 219483f
Coding Standards: Fix a few newly introduced WPCS issues.
SergeyBiryukov 75c3843
Options, Meta APIs: Improve logic to avoid unnecessary database write…
felixarntz 1e1ed26
Script Loader: Replace hardcoded output of style tags with calls to `…
spacedmonkey 76ea238
Editor: Introduce get_block_asset_url Utility Function.
spacedmonkey d87f9b8
HTML API: Remove all duplicate copies of an attribute when removing.
ockham 4e4b94a
Docs: Revert [56679].
desrosj a6bf745
Script Loader: Use `wp_get_script_tag()` and `wp_get_inline_script_ta…
westonruter 0eae815
Fonts: Get font-family name from 'fontFamily' field.
hellofromtonya ef7fa1c
Filesystem API: Introduce filters for before/after unzipping archives.
costdev 6bbce3a
Media: Rely on `wp_get_loading_optimization_attributes()` to add `dec…
felixarntz 670ab40
Filesystem API: Add missing ZIP file for unzip tests.
costdev f92ba90
Coding Standards: Fix a few newly introduced WPCS issues.
SergeyBiryukov 99a2b31
Media: Ensure images within shortcodes are correctly considered for l…
felixarntz 89f228b
Code Modernization: Rename parameters that use reserved keywords in `…
SergeyBiryukov 6e02a73
Coding Standards: Upgrade WPCS to version 3.0.0.
SergeyBiryukov b7614a2
Build/Test Tools: Use the latest PHP version for the coding standards…
SergeyBiryukov 5d0827c
Administration: Add event type to Events and News widget.
peterwilsoncc 42eb473
Editor: make better use of Tag Processor in layout block support.
tellthemachines 1ba9ebb
Build: Introduce SCRIPT_DEBUG global in webpack processing
gziolo 3b6b60f
Editor: remove null coalescing operator from layout.
tellthemachines cf48792
Results of npm run build:dev
mikachan 59c1099
Add sync pkg back to sourcemaps config
mikachan babbb7a
Add sync package to list of bundled packages
mikachan fce438a
Remove sync package from sourcemap config
mikachan 6854286
Results of npm run build:dev
mikachan ab68c96
Revert "Results of npm run build:dev"
mikachan 37c51de
Revert "Remove sync package from sourcemap config"
mikachan 6c77b49
Revert "Add sync package to list of bundled packages"
mikachan 27f9047
Revert "Add sync pkg back to sourcemaps config"
mikachan 5fa88d6
Revert "Results of npm run build:dev"
mikachan ee20e57
Revert "Editor: remove null coalescing operator from layout."
mikachan 1c1fe1b
Revert "Build: Introduce SCRIPT_DEBUG global in webpack processing"
mikachan 3333c8d
Revert "Editor: make better use of Tag Processor in layout block supp…
mikachan 2bec0bd
Revert "Administration: Add event type to Events and News widget."
mikachan 5d078a5
Revert "Build/Test Tools: Use the latest PHP version for the coding s…
mikachan 7a97b66
Revert "Coding Standards: Upgrade WPCS to version 3.0.0."
mikachan 78f9484
Revert "Code Modernization: Rename parameters that use reserved keywo…
mikachan abad93f
Revert "Media: Ensure images within shortcodes are correctly consider…
mikachan 2b1df6e
Revert "Coding Standards: Fix a few newly introduced WPCS issues."
mikachan bbff6d0
Revert "Filesystem API: Add missing ZIP file for unzip tests."
mikachan c8bd058
Revert "Media: Rely on `wp_get_loading_optimization_attributes()` to …
mikachan 43672e6
Revert "Filesystem API: Introduce filters for before/after unzipping …
mikachan 9c06e87
Revert "Fonts: Get font-family name from 'fontFamily' field."
mikachan de4ce4f
Revert "Script Loader: Use `wp_get_script_tag()` and `wp_get_inline_s…
mikachan 2237a5d
Revert "Docs: Revert [56679]."
mikachan bcb6413
Revert "HTML API: Remove all duplicate copies of an attribute when re…
mikachan 58235ef
Revert "Editor: Introduce get_block_asset_url Utility Function."
mikachan bdb938c
Revert "Script Loader: Replace hardcoded output of style tags with ca…
mikachan 0b307fa
Revert "Options, Meta APIs: Improve logic to avoid unnecessary databa…
mikachan 18ecc50
Revert "Coding Standards: Fix a few newly introduced WPCS issues."
mikachan 291019e
Revert "Docs: Correct typo in new `@since` tag."
mikachan 132c57c
Add sync package to list of bundled packages
mikachan 2ae8c6d
Results of npm run build:dev
mikachan 0587148
Cherry-pick "Results of npm run build:dev"
mikachan bb34412
Latest results from build:dev
mikachan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Update src/wp-includes/script-loader.php
Co-authored-by: Mukesh Panchal <[email protected]>
- Loading branch information
commit bac2694dafbc4bc3fc6c42658ba2959bde5caa0e
There are no files selected for viewing
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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@felixarntz, does this script need to define the
deferstrategy to ensure that view scripts depending on it don't fall back to the legacy script loading. They should all use the defer strategy after changes enabled by @westonruter.Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gziolo Indeed, that would be great! It's not "critical" since the dependents could still use
deferas they need to load later anyway, but I think it's a good practice to applydeferhere too. Similarly, we may then want to put this script into theheadinstead of the footer, so that the browser can detect it sooner.And we don't need to worry about breakage either, since if a script that depends on it doesn't use
defer, Core would set the interactivity script back to "notdefer" so that it would still work.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the
wp-interactivityscript was intended to be loaded in theheadwithdefer. Seegutenberg_interactivity_move_interactive_scripts_to_the_footer()in Gutenberg: https://github.com/WordPress/gutenberg/pull/52536/files#diff-2cfd8de00ffba84501209430457fc075fcffcd8fdc45bdcb0c41cfd2b28e7b2cL14-L26