Changes between Initial Version and Version 1 of Ticket #58632
- Timestamp:
- 06/29/2023 04:05:24 PM (3 years ago)
Legend:
- Unmodified
- Added
- Removed
- Modified
-
Ticket #58632 – Description
initial v1 3 3 While adding support for registering a script with an 'async' or 'defer' loading strategy, the solution that was committed [56033] did not include support for scripts that have an inline script attached in the 'after' position, nor if any script that is a dependent of the script has an inline script attached in the after position. In either scenario, the original script will be printed without the registered loading strategy in order to maintain execution order of the scripts in the dependency chain when an inline script is included. 4 4 5 The purpose of this ticket is to reopen discussion about adding support for this use case. An initial implementation that included support for this use case was included in [https://github.com/WordPress/wordpress-develop/pull/4391/ the initial PR] prior to removal in [https://github.com/WordPress/wordpress-develop/pull/4391/commits/b7d362df9655eedd2aff8651addd497fca83f1d0 this commit] . Relevant conversation on the initial ticket that led to the decision to remove support for inline scripts starts at [https://core.trac.wordpress.org/ticket/12009#comment:111 this comment].5 The purpose of this ticket is to reopen discussion about adding support for this use case. An initial implementation that included support for this use case was included in [https://github.com/WordPress/wordpress-develop/pull/4391/ the initial PR] prior to removal in [https://github.com/WordPress/wordpress-develop/pull/4391/commits/b7d362df9655eedd2aff8651addd497fca83f1d0 this commit] (from [https://github.com/10up/wordpress-develop/pull/81 squashed PR]). Relevant conversation on the initial ticket that led to the decision to remove support for inline scripts starts at [https://core.trac.wordpress.org/ticket/12009#comment:111 this comment].
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)